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.

Sign up for GitHub

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

Jump to bottom

SPEED_TO_MPH different values between Constants.cs and ApiConstants.cs #1363

Closed
originalfoo opened this issue Feb 6, 2022 · 0 comments · Fixed by #1364
Closed

SPEED_TO_MPH different values between Constants.cs and ApiConstants.cs #1363

originalfoo opened this issue Feb 6, 2022 · 0 comments · Fixed by #1364
Assignees
@originalfoo
Labels
BUG Defect detected confirmed Represents confirmed issue or bug high priority Affects lots of users SPEED LIMITS Feature: Speed limits
Milestone
11.6.4-hotfix-7

Comments

@originalfoo
Copy link
Member

Describe the problem

Different speeds between old constants and newer api constants....

Old:
image

API:
image

Related: #1224 #988

Also, the old constants should be [Obsolete].

PR incoming :)
@originalfoo originalfoo added BUG Defect detected high priority Affects lots of users confirmed Represents confirmed issue or bug SPEED LIMITS Feature: Speed limits labels Feb 6, 2022
@originalfoo originalfoo added this to the 11.6.4-hotfix-7 milestone Feb 6, 2022
@originalfoo originalfoo self-assigned this Feb 6, 2022
@originalfoo originalfoo mentioned this issue Feb 6, 2022
Merged
originalfoo added a commit that referenced this issue Feb 6, 2022
@originalfoo
What's New in TM:PE 11.6.4.7 ?
3a2faeb 
- [Meta] TM:PE 11.6.4-hotfix-7
- [Meta] Bugfix for default speeds which affects speed limits tool, overlays, and roundabout curvature speed
- [Fixed] Default netinfo speed should only inspect customisable lanes #1362 #1346 (aubergine18)
- [Fixed] Fix `SPEED_TO_MPH` value in `ApiConstants.cs` #1364 #1363 #988 (aubergine18)
- [Removed] Obsolete: `SPEED_TO_MPH` and `SPEED_TO_KMPH` in `Constants.cs` #1364 #1363 (aubergine18)
@originalfoo originalfoo mentioned this issue Feb 6, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@originalfoo originalfoo

Labels
BUG Defect detected confirmed Represents confirmed issue or bug high priority Affects lots of users SPEED LIMITS Feature: Speed limits
Projects
None yet
Milestone
11.6.4-hotfix-7
Development

Successfully merging a pull request may close this issue.

Fix SPEED_TO_MPH value in ApiConstants.cs CitiesSkylinesMods/TMPE
1 participant
@originalfoo