-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support of three windings transformers as variable or function in sensitivity analysis #654
Merged
Merged
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
387db41
Update.
annetill 347de8c
Merge branch 'main' into integration-core-5.1.0-SNAPSHOT
geofjamg de95802
Update
geofjamg 13e71d0
Wip
geofjamg ffc9959
Merge branch 'main' into integration-core-5.1.0-SNAPSHOT
geofjamg aeae3fa
Merge branch 'main' into integration-core-5.1.0-SNAPSHOT
annetill 009eb1b
Bump to 5.1.0-RC1.
annetill 8412409
Try supporting three windings transformers in sensitivity analysis.
obrix 25531af
Update value count.
obrix 969f2bc
More tests.
obrix b50c560
Test current function types.
obrix b50e3e8
Add test for phase variable with no phase shifter.
obrix f8cbd67
Small clean.
annetill 8312772
Some quick adjustments.
obrix ee65374
fix testThreeWindingsTransformerNoPhaseShifter.
obrix fbc67df
Save.
annetill b4d73b3
Fix code smell.
annetill 13e674c
Proposal.
annetill 72d4a29
Set core branch to sensi_indexing_by_variable_type and adapt.
obrix d2f9a93
Revert CI
flo-dup f0c493d
Adapt to ContingencyContext
flo-dup 055ff67
Fix unit test
flo-dup 07df711
Clean and add a unit test.
annetill af324d4
Clean.
annetill 72e1bdc
Merge branch 'main' into 3windingsTranformersInSensi
geofjamg 7e66967
Merge branch 'main' into 3windingsTranformersInSensi
annetill 2d5be0c
Fix code smells
flo-dup 631805d
Clean
geofjamg 50b3ec0
Clean
geofjamg 3611fc1
Small clean.
annetill File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would make sense to put that int (or Integer) inside the enum, don't you think?