-
Notifications
You must be signed in to change notification settings - Fork 85
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
Remove RushHour
code
#350
Comments
@aubergine10, Rush Hour still exists and can be subscribed to in the Workshop. This code seems to ensure the integration between Rush Hour's parking features and TM:PE's parking features. Real Time doesn't manage the vehicles at all, so this code has absolutely no effect on Real Time. |
That rush hour parking feature is what i ment in #298 |
Did Rush Hour II have those features? I thought it was only the older one which is kaput? |
Yes, you're right. Rush Hour II has no car parking logic anymore. Just checked it. |
Considering #298 we could convert it in to an option user interface or the global xml. Will pause this until after the harmony update is complete. |
As far as I remember I removed the Rush Hour compatibility code in the harmony branch |
Confirmed the obsolete code was removed as part of migration to harmony. |
There's a bunch of code relating to a mod,
RushHour
that no longer exists (and Rush Hour 2 is obsolete too).For example, in
GlobalConfig.cs
:https://github.com/krzychu124/Cities-Skylines-Traffic-Manager-President-Edition/blob/master/TLM/TLM/State/GlobalConfig.cs#L253-L280
That stuff can be ripped out as it's no longer relevant.
@dymanoid Is any of that relevant to Real Time?
The text was updated successfully, but these errors were encountered: