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
Add notebook example of asymmetric calculations. #482
Conversation
Signed-off-by: Zhen Wang <zhen.wang@alliander.com>
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.
will review the rest later today
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.
Just some minor comments on formality.
Signed-off-by: Zhen Wang <zhen.wang@alliander.com>
…idModel/power-grid-model into feature/asym_cal_example
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.
very good example. a couple nitpicks remaining
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.
My train is arriving at Utrecht, I'll submit this review for now and continue on my next train
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.
Second half
Signed-off-by: Zhen Wang <zhen.wang@alliander.com>
It's all looking good now, I have no complains. |
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.
only one nitpick
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.
Good work. I hereby approve the PR and set to auto-merge.
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
Fixes #447