Skip to content

Commit

Permalink
Merge pull request #251 from PowerGridModel/documentation/PGM-IO-dupl…
Browse files Browse the repository at this point in the history
…icated-excel-columns

Documentation/Vision export excel contains duplicated columns
  • Loading branch information
Jerry-Jinfeng-Guo authored May 8, 2024
2 parents 3d8eb4c + 300f6eb commit 8f8156f
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions docs/converters/vision_converter.md
Original file line number Diff line number Diff line change
Expand Up @@ -49,3 +49,11 @@ Some components are yet to be modeled for conversions because they might not hav
Vision introduced UUID based identifier system since version 9.7. It is implemented following the Microsoft naming scheme by replacing all the original identifier fields (i.e., '*Number', '*Subnumber', '*Nummber' and '*Subnummer') to GUID. This change brings the many benefits of UUIDs in general while on the other hand adds certain work on the conversion side. Since computations in PGM alone do not benefit from a string based identifier, we hence made the descision to perform UUID to integer conversion while maintaining the 'GUID' information in the `extra_info` field. Note that existing mapping files can still be used without significant changes, apart from adding `GUID` to the `extra` fields of interest.

An examplery usage can be found in the example notebook as well as in the test cases.

## Common/Known issues related to Vision
So far we have the following issue known to us related to Vision exported spread sheets. We provide a solution from user perspective to the best of our knowledge.

### Duplicated `P` columns
Vision can export sheets with duplicated `P` columns, one of which being unitless additional information. This field is of no actual purpose within PGM calculation.

**Tip:** We advice users to uncheck the `specifics` when exporting from Vision.

0 comments on commit 8f8156f

Please sign in to comment.