-
Notifications
You must be signed in to change notification settings - Fork 5
Resolving issue #378- Check ValidationUtils for negative Susceptance #499
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
Resolving issue #378- Check ValidationUtils for negative Susceptance #499
Conversation
… are allowed for the transformer susceptance.
Closes #378 |
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.
Please additionally check the CI results. Currently some tests are broken.
src/main/java/edu/ie3/datamodel/utils/validation/ConnectorValidationUtils.java
Outdated
Show resolved
Hide resolved
src/main/java/edu/ie3/datamodel/utils/validation/ConnectorValidationUtils.java
Outdated
Show resolved
Hide resolved
… of https://github.com/ie3-institute/PowerSystemDataModel into uf/#378-Check-ValidationUtils-for-negative-Susceptance
This comment has been minimized.
This comment has been minimized.
Codecov Report
@@ Coverage Diff @@
## dev #499 +/- ##
============================================
- Coverage 78.89% 78.88% -0.01%
+ Complexity 2107 2106 -1
============================================
Files 263 263
Lines 8273 8275 +2
Branches 780 780
============================================
+ Hits 6527 6528 +1
- Misses 1345 1350 +5
+ Partials 401 397 -4
Continue to review full report at Codecov.
|
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.
Generally looks good! Only some minor change requests regarding the documentation :-)
src/main/java/edu/ie3/datamodel/utils/validation/ConnectorValidationUtils.java
Outdated
Show resolved
Hide resolved
src/main/java/edu/ie3/datamodel/utils/validation/ConnectorValidationUtils.java
Outdated
Show resolved
Hide resolved
src/main/java/edu/ie3/datamodel/utils/validation/ConnectorValidationUtils.java
Outdated
Show resolved
Hide resolved
src/main/java/edu/ie3/datamodel/utils/validation/ValidationUtils.java
Outdated
Show resolved
Hide resolved
…dationUtils.java Co-authored-by: Chris Kittl <[email protected]>
…dationUtils.java Co-authored-by: Chris Kittl <[email protected]>
…ls.java Co-authored-by: Chris Kittl <[email protected]>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Analysis Details0 IssuesCoverage and DuplicationsProject ID: edu.ie3:PowerSystemDataModel |
Adapting ValidationUtils and test data such that only negative values are allowed for the transformer susceptance.