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

Clean cgmes-only parameters from LayoutParameters #510

Open
So-Fras opened this issue Mar 29, 2023 · 0 comments
Open

Clean cgmes-only parameters from LayoutParameters #510

So-Fras opened this issue Mar 29, 2023 · 0 comments

Comments

@So-Fras
Copy link
Collaborator

So-Fras commented Mar 29, 2023

  • Do you want to request a feature or report a bug?
    Feature

  • What is the current behavior?
    In single-line diagram, some cgmes-only parameters (cgmesUseName, cgmesScaleFactor, cgmesDiagramName) are present in LayoutParameters but LayoutParameters is not the right class to contain them.
    The rationale behind this was that the smart selector did not allow to pass any parameters, so the LayoutParameters class was used for cgmes-only parameters.

  • What is the expected behavior?
    Some refactoring is necessary to clean this. Cgmes-only parameters should not be stored in the LayoutParameters class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant