Skip to content
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

Feature addition to check if p_sigma and q_sigma co-(in)exist. #425

Conversation

Jerry-Jinfeng-Guo
Copy link
Contributor

@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo commented Nov 10, 2023

Fixes issue: # name and number of the issue

Draft PR

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
…revent KeyError

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
src/power_grid_model/validation/rules.py Outdated Show resolved Hide resolved
src/power_grid_model/validation/rules.py Outdated Show resolved Hide resolved
tests/unit/validation/test_validation_functions.py Outdated Show resolved Hide resolved
src/power_grid_model/validation/rules.py Outdated Show resolved Hide resolved
@Jerry-Jinfeng-Guo
Copy link
Contributor Author

How on the actual earth can I get rid of the Check Code Quality error? 💢

…ified logic checking `p_sigma` and `q_sigma`.

Also bumped reuse version.

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo marked this pull request as ready for review November 17, 2023 15:56
…air. Included parameterized test cases that enumerates all possible combinations.

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
updated the doc: fixed the markdown link

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
docs/user_manual/components.md Show resolved Hide resolved
src/power_grid_model/validation/rules.py Outdated Show resolved Hide resolved
src/power_grid_model/validation/rules.py Outdated Show resolved Hide resolved
tests/unit/validation/test_validation_functions.py Outdated Show resolved Hide resolved
…ensor`; added test cases for data containing both components, `sym_power_sensor` and `asym_power_sensor`.

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
…); fixed `any()`; error ids more explicitly verified;

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Copy link
Member

@mgovers mgovers left a comment

Choose a reason for hiding this comment

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

i think it's pretty much done. just a couple final nitpicks.

please also don't forget to update the branch with origin/main (either locally or using the Update Branch button in the PR

src/power_grid_model/validation/rules.py Outdated Show resolved Hide resolved
tests/unit/validation/test_validation_functions.py Outdated Show resolved Hide resolved
tests/unit/validation/test_validation_functions.py Outdated Show resolved Hide resolved
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
@TonyXiang8787 TonyXiang8787 merged commit 3e12077 into PowerGridModel:main Nov 24, 2023
25 checks passed
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo self-assigned this Feb 2, 2024
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.

None yet

3 participants