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

Fix the naming of CalculatedBus and MergedBus #54

Closed
mathbagu opened this issue Mar 13, 2020 · 1 comment · Fixed by #56
Closed

Fix the naming of CalculatedBus and MergedBus #54

mathbagu opened this issue Mar 13, 2020 · 1 comment · Fixed by #56
Assignees
Labels
bug Something isn't working

Comments

@mathbagu
Copy link

  • Do you want to request a feature or report a bug?
    Bug
    Fix calculated/merged bus name powsybl-core#1148

  • What is the current behavior?
    The name of the CalculatedBus and the MergedBus is the same than the ID.

  • If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

  • What is the expected behavior?
    The name of the bus should be based on the name of the VoltageLevel.

  • What is the motivation / use case for changing the behavior?
    Have both Java and C++ implementation as close as possible.

  • Please tell us about your environment:

    • PowSyBl Version: ...
    • OS Version: ...
  • Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, spectrum, etc)

(if a question doesn't apply, you can delete it)

@mathbagu mathbagu added the bug Something isn't working label Mar 13, 2020
@mathbagu mathbagu self-assigned this Mar 13, 2020
@mathbagu
Copy link
Author

Duplicate #31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant