Skip to content

Add explicit size property to FancyArray#225

Merged
jaapschoutenalliander merged 7 commits intoPowerGridModel:mainfrom
TomvdPoll:feature/add-explicit-size-to-array
Apr 13, 2026
Merged

Add explicit size property to FancyArray#225
jaapschoutenalliander merged 7 commits intoPowerGridModel:mainfrom
TomvdPoll:feature/add-explicit-size-to-array

Conversation

@TomvdPoll
Copy link
Copy Markdown
Contributor

Fixes #221

Changes proposed in this PR include:

  • Add the size property explicitly to the FancyArray class, so that static type checkers can infer the property type

Could you please pay extra attention to the points below when reviewing the PR:

  • Not sure what the testing strategy is for the repo, but I added a simple test to ensure .size works

I, TomvandePoll <tom.van.de.poll@alliander.com>, hereby add my Signed-off-by to this commit: c287ddd

Signed-off-by: TomvandePoll <tom.van.de.poll@alliander.com>
Signed-off-by: TomvandePoll <tom.van.de.poll@alliander.com>
I, TomvandePoll <tom.van.de.poll@alliander.com>, hereby add my Signed-off-by to this commit: 8acebc2
I, TomvandePoll <tom.van.de.poll@alliander.com>, hereby add my Signed-off-by to this commit: fd2148e

Signed-off-by: TomvandePoll <tom.van.de.poll@alliander.com>
Copy link
Copy Markdown
Member

@jaapschoutenalliander jaapschoutenalliander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, @vincentkoppen ok with this as well?

@jaapschoutenalliander jaapschoutenalliander merged commit 4088d7d into PowerGridModel:main Apr 13, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Pyright has trouble infering the type of an array's .size property

3 participants