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

[BUG] Inconsistant units possible between line and linecode? #469

Open
John-Boik opened this issue Jul 10, 2024 · 1 comment
Open

[BUG] Inconsistant units possible between line and linecode? #469

John-Boik opened this issue Jul 10, 2024 · 1 comment

Comments

@John-Boik
Copy link
Contributor

In create_dss_object, for linecodes, several lines similar to the following appear:

    linecode.r1 = linecode.r1 / _convert_to_meters[linecode.units]
    linecode.x1 = linecode.x1 / _convert_to_meters[linecode.units]

Similar equations are also used when creating lines. Apparently in PMD, if units is not given for a linecode, then the multiplier defaults to 1.0 and a units=m is set for the linecode. But when units is not given for linecode, DSS apparently looks to the definition of units on lines that use the linecode, to determine what the units for linecode should be. Isn't it true that in the PMD parser, there would be an inconsistency in values if units for linecode is not given, but is given for lines that use the linecode, and these units are not meters?

If so, part of the solution might be to search for all lines that use a linecode when creating the linecode and units is not given. If all such lines have at most one unique value for units, then that is the value that the linecode should have. If lines are created after linecodes, then the search and any units changes might be more complicated.

Copy link

stale bot commented Jan 9, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label Jan 9, 2025
@stale stale bot closed this as completed Jan 18, 2025
@pseudocubic pseudocubic reopened this Jan 21, 2025
@pseudocubic pseudocubic added Type: Bug Something isn't working Category: Data Parsers parser issues pinned Status: Help Wanted and removed wontfix This will not be worked on labels Jan 21, 2025
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

2 participants