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

Ugly CGMES DL diagram #5

Closed
geofjamg opened this issue Oct 7, 2019 · 3 comments
Closed

Ugly CGMES DL diagram #5

geofjamg opened this issue Oct 7, 2019 · 3 comments
Assignees

Comments

@geofjamg
Copy link
Member

geofjamg commented Oct 7, 2019

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

  • What is the current behavior?
    Here is attached sample.zip, a small 1 substation, 3 voltage level CGMES file, extracted from real data. sample.zip

Using automatic layout without extensions and vertical substation layout we can generate this diagram:

image

The following code generates using same layout configuration a DL file.

Network network = Importers.loadNetwork("sample.zip");

new LayoutToCgmesExtensionsConverter(new VerticalSubstationLayoutFactory(), new PositionVoltageLevelLayoutFactory(new PositionFree()), new LayoutParameters(), true)
    .convertLayout(network);

new CgmesDLExporter(network)
    .exportDLData(new FileDataSource(Paths.get("/tmp"), "sample"));

Then after adding sample_DL.xml to the original zip, we can generate this diagram using CGMES voltage and substation layout:

image

I seems that there is a lot of missing data and diagram is not really usable.
Did I missed something in the way I generated the data?
Maybe we introduced a regression in last PR?

@CBiasuzzi
Copy link
Contributor

I'll investigate. However, Importers complains about this sample.zip network (com.powsybl.cgmes.model.CgmesModelException: nominalVoltage not found for ...).
Could it be possible to have a powsybl-working version?

@geofjamg
Copy link
Member Author

geofjamg commented Oct 8, 2019

I'll investigate. However, Importers complains about this sample.zip network (com.powsybl.cgmes.model.CgmesModelException: nominalVoltage not found for ...).
Could it be possible to have a powsybl-working version?

Sorry, I uploaded a new version. Is it better?

@CBiasuzzi
Copy link
Contributor

thx: no import exceptions, with this one.

@geofjamg geofjamg transferred this issue from powsybl/powsybl-incubator Oct 14, 2019
@CBiasuzzi CBiasuzzi mentioned this issue Oct 30, 2019
5 tasks
flo-dup added a commit that referenced this issue Jun 12, 2023
Signed-off-by: Florian Dupuy <florian.dupuy@rte-france.com>
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

No branches or pull requests

3 participants