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

Change any OSM length units from cm to m #183

Merged
merged 2 commits into from
Jun 29, 2023

Conversation

Robbendebiene
Copy link
Member

Resolves #128

Explanation:

Inputs | PAD  | INSERT | REPLACE
1      > 001  > 0.01   > 0.01
1000   > 1000 > 10.00  > 10
12     > 012  > 0.12   > 0.12
0      > 000  > 0.00   > 0
102    > 102  > 1.02   > 1.02
120    > 120  > 1.20   > 1.2

Note:

  • This expression combination only works for positive integers (not for negative or decimal numbers) and does a conversion by exactly two decimal places to the left.
  • This PR changes the platform width unit from centimeters to meters, mainly because it seems more reasonable. Counter argument to do this, could be that so far all length units use centimeter so users could expect centimeters after all. However the min/max restrictions should at least prevent wrong input values.

@Robbendebiene Robbendebiene merged commit e89aa19 into more_expressions Jun 29, 2023
3 of 5 checks passed
@Robbendebiene Robbendebiene deleted the migrate-centimeter-to-meter branch June 29, 2023 06:31
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

Successfully merging this pull request may close these issues.

Enter width of platforms in the format that is standard for OSM (comma-seperated meters)
2 participants