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

RCT1 Park Value Inconsistency #5253

Closed
2 tasks
Christian-Seematter opened this issue Feb 23, 2017 · 7 comments
Closed
2 tasks

RCT1 Park Value Inconsistency #5253

Christian-Seematter opened this issue Feb 23, 2017 · 7 comments
Assignees
Labels
rct1 Relates to RollerCoaster Tycoon 1.

Comments

@Christian-Seematter
Copy link

OS: Windows 10
Version: 0.0.7
Commit/Build: 7005d5d

This solution in #5218 does not resolve the inconsistency issues. It be better to have a compatibility mode for .sv4 files that calculates it the same way as RCT1.

I tried loading my save games to unlock the scenarios from RCT1 in openRCT2 that I had just replayed in the last couple of days, but my savegames that passed the requirements in terms of park value do not pass in openRCT2.

  • Reproducible in RCT2 (vanilla)?
  • Multiplayer?

Screenshots / Video:
2017-02-23 14_36_10-openrct2

Save game:
If it's ok to post scenarios from RCT1 I'll gladly post the one in the screenshot here.

@Gymnasiast
Copy link
Member

It be better to have a compatibility mode for .sv4 files that calculates it the same way as RCT1.

Easier said than done.

If it's ok to post scenarios from RCT1 I'll gladly post the one in the screenshot here.

Um, yes, we'd need those. Especially if you have scenario that barely passes in RCT1.

@Christian-Seematter
Copy link
Author

Christian-Seematter commented Feb 23, 2017

Yeah, it does sound more like part of an Epic; "RCT1 Support". Is there somebody on the team who already knows about the differences on how they are calculated? If it does scale linearly, then a factor of 10 was just a bit to high. The ratio in #5218 was around 7.7 if I remember. I might experiment with it a bit in a fork.

Diamond Heights_26.zip

@duncanspumpkin
Copy link
Contributor

@Christian-Seematter it would mean reimplementing RCT1 from almost scratch, it would be a massive undertaking.

@Christian-Seematter
Copy link
Author

The other metrics already line up fine, don't they? Worst case scenario a tweak of the scaling factor we currently have set to 10 will make the old scenarios well playable again.

I'm not trying to push anything here. Just found this project and am excited about how far it's already gotten. You guys are doing a great job! Since there is a github label for "RCT1 Import" I concluded that that is one of the goals too.

@duncanspumpkin
Copy link
Contributor

Yeah, by import we plan to support rct1 files but play them based on rct2 mechanics

@IntelOrca
Copy link
Contributor

I think lowering the multiplier would be better.

@Gymnasiast Gymnasiast added the rct1 Relates to RollerCoaster Tycoon 1. label Feb 26, 2017
@Gymnasiast
Copy link
Member

I plan to lower the multiplier to 7.5. Based on the parks I have seen, it's a little bit lower than the "actual"* difference, but that at least allows parks that passed in RCT1 to also pass in OpenRCT2.

* Put between quotes since the different mechanics mean that any value will be off for some parks, at least a small amount.

@Gymnasiast Gymnasiast self-assigned this May 12, 2017
@janisozaur janisozaur changed the title RCT1 Park Value Inconscistency RCT1 Park Value Inconsistency May 13, 2017
Gymnasiast added a commit to Gymnasiast/OpenRCT2 that referenced this issue Jun 29, 2017
Gymnasiast added a commit to Gymnasiast/OpenRCT2 that referenced this issue Jun 29, 2017
IntelOrca added a commit that referenced this issue Jun 30, 2017
Fix #5253: RCT1 park value conversion factor too high
janisozaur added a commit that referenced this issue Jul 12, 2017
- Feature: [#1399 (partial), #5177] Add window that displays any missing/corrupt objects when loading a park
- Feature: [#5056] Add cheat to own all land.
- Feature: [#5133] Add option to display guest expenditure (as seen in RCTC).
- Feature: [#5196] Add cheat to disable ride ageing.
- Feature: [#5504] Group vehicles into ride groups
- Feature: [#5576] Add a persistent 'display real names of guests' setting.
- Feature: [#5611] Add support for Android
- Feature: [#5706] Add support for OpenBSD
- Feature: OpenRCT2 now starts up on the display it was last shown on.
- Feature: Park entrance fee can now be set to amounts up to £200.
- Improved: Construction rights can now be placed on park entrances.
- Improved: Mouse can now be dragged to select scenery when saving track designs
- Fix: [#259] Money making glitch involving swamps (original bug)
- Fix: [#441] Construction rights over entrance path erased (original bug)
- Fix: [#578] Ride ghosts show up in ride list during construction (original bug)
- Fix: [#597] 'Finish 5 roller coasters' goal not properly checked (original bug)
- Fix: [#667] Incorrect banner limit calculation (original bug)
- Fix: [#739] Crocodile Ride (Log Flume) never allows more than five boats (original bug)
- Fix: [#837] Can't move windows on title screen to where the toolbar would be (original bug)
- Fix: [#1705] Time Twister's Medieval entrance has incorrect scrolling (original bug)
- Fix: [#3178, #5456] Paths with non-ASCII characters not handled properly on macOS.
- Fix: [#3346] Crash when extra long train breaks down at the back
- Fix: [#3479] Building in pause mode creates too many floating numbers, crashing the game
- Fix: [#3565] Multiplayer server crash
- Fix: [#3681] Steel Twister rollercoaster always shows all track designs
- Fix: [#3846, #5749] Crash when testing coaster with a diagonal lift in block brake mode
- Fix: [#4054] Sorting rides by track type: Misleading research messages
- Fix: [#4055] Sort rides by track type: Sorting rule is not really clear (inconsistent?)
- Fix: [#4512] Invisible map edge tiles corrupted
- Fix: [#5009] Ride rating calculations can overflow
- Fix: [#5253] RCT1 park value conversion factor too high
- Fix: [#5400] New Ride window does not focus properly on newly invented ride.
- Fix: [#5489] Sprite index crash for car view on car ride.
- Fix: [#5730] Unable to uncheck 'No money' in the Scenario Editor.
- Fix: [#5750] Game freezes when ride queue linked list is corrupted.
- Fix: [#5819] Vertical multi-dimension coaster tunnels drawn incorrectly
- Fix: Non-invented vehicles can be used via track designs in select-by-track-type mode.
- Fix: Track components added by OpenRCT2 are now usable in older scenarios.
- Technical: [#5047] Add ride ratings tests
- Technical: [#5458] Begin offering headless build with reduced compile- and run-time dependencies
- Technical: [#5755] Title sequence wait periods use milliseconds
- Technical: Fix many desync sources
ZehMatt pushed a commit to ZehMatt/OpenRCT2 that referenced this issue Jul 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rct1 Relates to RollerCoaster Tycoon 1.
Projects
None yet
Development

No branches or pull requests

4 participants