-
Notifications
You must be signed in to change notification settings - Fork 20
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
Fix typeerror exception when importing dynamic data of synchronous machines #24
base: master
Are you sure you want to change the base?
Fix typeerror exception when importing dynamic data of synchronous machines #24
Conversation
817aeb2
to
8d937f0
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
8d937f0
to
26d6b7b
Compare
@martinmoraga could you please add the dco sign off? Then this can be merged. |
Has this been fixed in CIMgen? Fixing it just here wont help anybody in the long run as the changes will get lost when the code gets regenerated. |
I agree. My intention was to create an issue in cimgen when merging this so that we don’t forget this issue. |
26d6b7b
to
bc81cc6
Compare
initialize dynamic parameters of class SynchronousMachineTimeConstantReactance as float Signed-off-by: Martin Moraga <[email protected]> Signed-off-by: Martin Moraga <[email protected]>
initialize parameter 'inertia' of class RotatingMachineDynamics as float Signed-off-by: Martin Moraga <[email protected]> Signed-off-by: Martin Moraga <[email protected]>
bc81cc6
to
6c33b53
Compare
Quality Gate passedIssues Measures |
@m-mirz Done! But the pipeline fails now with the error:
|
Okay, thats my fault. Sorry I will have a look |
Issue #23