Skip to content

Add new track pieces to the Twister, Hyper Twister and Vertical Drop Roller Coasters #23051

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

Merged
merged 12 commits into from
Nov 11, 2024

Conversation

mixiate
Copy link
Contributor

@mixiate mixiate commented Oct 25, 2024

This adds new track pieces to the twister, hyper twister and vertical drop rollercoasters.

  • Large unbanked and banked sloped curves (Twister, Hyper, Vertical Drop)
  • Large corkscrews (Twister)
  • Medium half loops (Twister, Vertical Drop)
  • Small and large zero g rolls (Cheats)

newtwisterpieces

All pieces have been tested with the applicable trains. The zero g rolls are currently cheats only as they don't seem to currently be optionally available only for trains that have the sprites. The trains don't have sprites for all the diagonal banking pieces either but these are already available for the giga coaster so I just made them available to these too. Maybe both of these things could be sorted out eventually.

@mixiate mixiate force-pushed the extended-twister-roller-coaster branch from 4a22540 to b64bd78 Compare October 25, 2024 20:38
@SpartanFrederic104
Copy link
Contributor

Did some testing with these new track pieces and they just look fantastic and smooth as usual.
Fred's Workbench 2024-10-25 13-47-09
Fred's Workbench 2024-10-25 13-47-15
Fred's Workbench 2024-10-25 13-47-31
I haven't found any major issues so far (well, excluding the vanilla trains glitching on some diagonal banked slope pieces and the lack of sprites for zero-G rolls). Can't wait to play with these new track pieces in a future release build!
(and yes these screenshots were taken before the force push was done)

@mixiate
Copy link
Contributor Author

mixiate commented Oct 25, 2024

Thanks for testing them again, I really appreciate it. I just fixed a few pieces that didn't connect quite right, and a couple of supports that didn't reach, and rebased it. Hopefully this is all okay now.

@fidwell
Copy link
Contributor

fidwell commented Oct 25, 2024

These look great. The corkscrews are even better looking than the originals. Nice work!

Dorney Park   Wildwater Kingdom 2024-10-25 18-46-27

@karst
Copy link
Member

karst commented Oct 25, 2024

Thanks for taking in my feedback regarding the polygon spines :)
Looks lovely!

@karst karst added testing required PR needs to be tested before it is merged. feature Add something new to the game. network version Network version needs updating - double check before merging! park file version Requires updating the park file version number labels Oct 25, 2024
@karst
Copy link
Member

karst commented Oct 25, 2024

There's a few more things required before this can be merged:

@mixiate mixiate force-pushed the extended-twister-roller-coaster branch 2 times, most recently from 188a5b5 to 4ce4c76 Compare November 2, 2024 21:55
@SpartanFrederic104
Copy link
Contributor

Guessing this will need a rebase and the changelog entry moved to the v0.4.17 section?

@mixiate
Copy link
Contributor Author

mixiate commented Nov 5, 2024

Yep, i'll sort this out as soon as I can.

@mixiate mixiate force-pushed the extended-twister-roller-coaster branch from 4ce4c76 to eb84266 Compare November 6, 2024 02:59
@Gymnasiast Gymnasiast added this to the v0.4.17 milestone Nov 8, 2024
@Gymnasiast Gymnasiast mentioned this pull request Nov 8, 2024
.enabledTrackGroups = {TrackGroup::flat, TrackGroup::straight, TrackGroup::stationEnd, TrackGroup::liftHill, TrackGroup::liftHillSteep, TrackGroup::flatRollBanking, TrackGroup::verticalLoop, TrackGroup::slope, TrackGroup::slopeSteepUp, TrackGroup::slopeSteepDown, TrackGroup::flatToSteepSlope, TrackGroup::slopeCurve, TrackGroup::slopeCurveSteep, TrackGroup::sBend, TrackGroup::curveSmall, TrackGroup::curve, TrackGroup::curveLarge, TrackGroup::helixDownBankedHalf, TrackGroup::helixUpBankedHalf, TrackGroup::brakes, TrackGroup::onridePhoto, TrackGroup::slopeVertical, TrackGroup::slopeCurveBanked, TrackGroup::blockBrakes, TrackGroup::slopeRollBanking, TrackGroup::curveVertical, TrackGroup::halfLoopLarge, TrackGroup::brakeForDrop, TrackGroup::diagBrakes, TrackGroup::diagBlockBrakes},
.extraTrackGroups = {TrackGroup::halfLoop, TrackGroup::corkscrew, TrackGroup::barrelRoll, TrackGroup::poweredLift, TrackGroup::halfLoopLarge, TrackGroup::quarterLoop, TrackGroup::booster, TrackGroup::twist},
.enabledTrackGroups = {TrackGroup::flat, TrackGroup::straight, TrackGroup::stationEnd, TrackGroup::liftHill, TrackGroup::liftHillSteep, TrackGroup::flatRollBanking, TrackGroup::verticalLoop, TrackGroup::slope, TrackGroup::slopeSteepUp, TrackGroup::slopeSteepDown, TrackGroup::flatToSteepSlope, TrackGroup::slopeCurve, TrackGroup::slopeCurveSteep, TrackGroup::sBend, TrackGroup::curveSmall, TrackGroup::curve, TrackGroup::curveLarge, TrackGroup::helixDownBankedHalf, TrackGroup::helixUpBankedHalf, TrackGroup::brakes, TrackGroup::onridePhoto, TrackGroup::slopeVertical, TrackGroup::slopeCurveBanked, TrackGroup::blockBrakes, TrackGroup::slopeRollBanking, TrackGroup::curveVertical, TrackGroup::halfLoopLarge, TrackGroup::brakeForDrop, TrackGroup::diagBrakes, TrackGroup::diagBlockBrakes, TrackGroup::slopeCurveLarge, TrackGroup::halfLoopMedium},
.extraTrackGroups = {TrackGroup::halfLoop, TrackGroup::corkscrew, TrackGroup::barrelRoll, TrackGroup::poweredLift, TrackGroup::halfLoopLarge, TrackGroup::quarterLoop, TrackGroup::booster, TrackGroup::twist, TrackGroup::zeroGRoll, TrackGroup::zeroGRollLarge, TrackGroup::corkscrewLarge},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a particular reason to lock the Zero G Rolls behind a cheat?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

imo there isn't, if a vehicle can't draw the pieces they'll be disabled anyhow, and the vertical drop coaster is known to have this element, especially immelmen.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Originally it was because none of the new inversions had entries for the sprites required, but that has now been fixed. I added the zero g rolls to the twister but forgot about updating these for the vertical drop coaster. Should the large corkscrew also be available?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah definitely

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes and i made a lot of BM Cars now but i think we need an permission to make them official?

Copy link
Member

@Gymnasiast Gymnasiast Nov 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mixiate Ok, could you apply that and rebase?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. I've checked that the park and network version are okay also.

@mixiate mixiate force-pushed the extended-twister-roller-coaster branch from eb84266 to 2e591a7 Compare November 10, 2024 21:18
@karst
Copy link
Member

karst commented Nov 10, 2024

As far as I can see the code looks all good. I'll have a dev look at it tho.
I'll give this a playtest.

Copy link
Member

@karst karst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonderful :) After some playing with it, all items seem to be working correctly. Conversion also works

@Gymnasiast Gymnasiast merged commit fd90afe into OpenRCT2:develop Nov 11, 2024
23 checks passed
case TrackElemType::LeftLargeHalfLoopUp:
case TrackElemType::RightLargeHalfLoopUp:
case TrackElemType::LeftLargeHalfLoopDown:
case TrackElemType::RightLargeHalfLoopDown:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These four lines need to be removed. Sending a patch right away.

janisozaur added a commit that referenced this pull request Dec 8, 2024
- Feature: [#23166] Add Galician translation.
- Feature: [#23227] Add Classic Wooden Twister roller coaster, for better compatibility with RCT1.
- Improved: [#23051] Add large sloped turns and new inversions to the Twister, Vertical Drop, Hyper and Flying Roller Coasters.
- Improved: [#23123] Improve sorting of roller coasters in build new ride menu.
- Improved: [#23211] Add boosters to classic wooden roller coaster (cheats only).
- Improved: [#23229] Add debug option for making the sprite sorting algorithm stable.
- Improved: [#23233] Add diagonal booster to LSM Launched Coaster.
- Improved: [#23277] Add small, medium and large turns, diagonal track and S-bends to the Boat Hire.
- Fix: [#20070, #22972] Missing and mismatched flat and sloped footpaths on several scenarios.
- Fix: [#22726] ‘Force park rating’ cheat is not saved with the park.
- Fix: [#23064] Stand-Up Roller Coaster unbanked to banked track pieces are misaligned.
- Fix: [#23066] Stand-Up Roller Coaster has many supports that don't join up to the track.
- Fix: [#23206] Multiplayer desyncs when FPS is uncapped.
- Fix: [#23238] Updating a guest’s favourite ride works differently from vanilla RCT2.
- Fix: [objects#355] Fix colour preset settings of the Stand-Up Roller Coaster trains.
- Fix: [objects#355] Fix colour preset settings of many vehicles from Wacky Worlds and Time Twister.
janisozaur added a commit that referenced this pull request Dec 8, 2024
- Feature: [#23166] Add Galician translation.
- Feature: [#23227] Add Classic Wooden Twister roller coaster, for better compatibility with RCT1.
- Improved: [#23051] Add large sloped turns and new inversions to the Twister, Vertical Drop, Hyper and Flying Roller Coasters.
- Improved: [#23123] Improve sorting of roller coasters in build new ride menu.
- Improved: [#23211] Add boosters to classic wooden roller coaster (cheats only).
- Improved: [#23229] Add debug option for making the sprite sorting algorithm stable.
- Improved: [#23233] Add diagonal booster to LSM Launched Coaster.
- Improved: [#23277] Add small, medium and large turns, diagonal track and S-bends to the Boat Hire.
- Fix: [#20070, #22972] Missing and mismatched flat and sloped footpaths on several scenarios.
- Fix: [#22726] ‘Force park rating’ cheat is not saved with the park.
- Fix: [#23064] Stand-Up Roller Coaster unbanked to banked track pieces are misaligned.
- Fix: [#23066] Stand-Up Roller Coaster has many supports that don't join up to the track.
- Fix: [#23206] Multiplayer desyncs when FPS is uncapped.
- Fix: [#23238] Updating a guest’s favourite ride works differently from vanilla RCT2.
- Fix: [objects#355] Fix colour preset settings of the Stand-Up Roller Coaster trains.
- Fix: [objects#355] Fix colour preset settings of many vehicles from Wacky Worlds and Time Twister.
janisozaur added a commit that referenced this pull request Dec 8, 2024
- Feature: [#23166] Add Galician translation.
- Feature: [#23227] Add Classic Wooden Twister roller coaster, for better compatibility with RCT1.
- Improved: [#23051] Add large sloped turns and new inversions to the Twister, Vertical Drop, Hyper and Flying Roller Coasters.
- Improved: [#23123] Improve sorting of roller coasters in build new ride menu.
- Improved: [#23211] Add boosters to classic wooden roller coaster (cheats only).
- Improved: [#23229] Add debug option for making the sprite sorting algorithm stable.
- Improved: [#23233] Add diagonal booster to LSM Launched Coaster.
- Improved: [#23277] Add small, medium and large turns, diagonal track and S-bends to the Boat Hire.
- Fix: [#20070, #22972] Missing and mismatched flat and sloped footpaths on several scenarios.
- Fix: [#22726] ‘Force park rating’ cheat is not saved with the park.
- Fix: [#23064] Stand-Up Roller Coaster unbanked to banked track pieces are misaligned.
- Fix: [#23066] Stand-Up Roller Coaster has many supports that don't join up to the track.
- Fix: [#23206] Multiplayer desyncs when FPS is uncapped.
- Fix: [#23238] Updating a guest’s favourite ride works differently from vanilla RCT2.
- Fix: [objects#355] Fix colour preset settings of the Stand-Up Roller Coaster trains.
- Fix: [objects#355] Fix colour preset settings of many vehicles from Wacky Worlds and Time Twister.
janisozaur added a commit that referenced this pull request Dec 8, 2024
- Feature: [#23166] Add Galician translation.
- Feature: [#23227] Add Classic Wooden Twister roller coaster, for better compatibility with RCT1.
- Improved: [#23051] Add large sloped turns and new inversions to the Twister, Vertical Drop, Hyper and Flying Roller Coasters.
- Improved: [#23123] Improve sorting of roller coasters in build new ride menu.
- Improved: [#23211] Add boosters to classic wooden roller coaster (cheats only).
- Improved: [#23229] Add debug option for making the sprite sorting algorithm stable.
- Improved: [#23233] Add diagonal booster to LSM Launched Coaster.
- Improved: [#23277] Add small, medium and large turns, diagonal track and S-bends to the Boat Hire.
- Fix: [#20070, #22972] Missing and mismatched flat and sloped footpaths on several scenarios.
- Fix: [#22726] ‘Force park rating’ cheat is not saved with the park.
- Fix: [#23064] Stand-Up Roller Coaster unbanked to banked track pieces are misaligned.
- Fix: [#23066] Stand-Up Roller Coaster has many supports that don't join up to the track.
- Fix: [#23206] Multiplayer desyncs when FPS is uncapped.
- Fix: [#23238] Updating a guest’s favourite ride works differently from vanilla RCT2.
- Fix: [objects#355] Fix colour preset settings of the Stand-Up Roller Coaster trains.
- Fix: [objects#355] Fix colour preset settings of many vehicles from Wacky Worlds and Time Twister.
CorySanin added a commit to CorySanin/OpenRCT2 that referenced this pull request Feb 3, 2025
Release v0.4.17

- Feature: [OpenRCT2#23166] Add Galician translation.
- Feature: [OpenRCT2#23227] Add Classic Wooden Twister roller coaster, for better compatibility with RCT1.
- Improved: [OpenRCT2#23051] Add large sloped turns and new inversions to the Twister, Vertical Drop, Hyper and Flying Roller Coasters.
- Improved: [OpenRCT2#23123] Improve sorting of roller coasters in build new ride menu.
- Improved: [OpenRCT2#23211] Add boosters to classic wooden roller coaster (cheats only).
- Improved: [OpenRCT2#23229] Add debug option for making the sprite sorting algorithm stable.
- Improved: [OpenRCT2#23233] Add diagonal booster to LSM Launched Coaster.
- Improved: [OpenRCT2#23277] Add small, medium and large turns, diagonal track and S-bends to the Boat Hire.
- Fix: [OpenRCT2#20070, OpenRCT2#22972] Missing and mismatched flat and sloped footpaths on several scenarios.
- Fix: [OpenRCT2#22726] ‘Force park rating’ cheat is not saved with the park.
- Fix: [OpenRCT2#23064] Stand-Up Roller Coaster unbanked to banked track pieces are misaligned.
- Fix: [OpenRCT2#23066] Stand-Up Roller Coaster has many supports that don't join up to the track.
- Fix: [OpenRCT2#23206] Multiplayer desyncs when FPS is uncapped.
- Fix: [OpenRCT2#23238] Updating a guest’s favourite ride works differently from vanilla RCT2.
- Fix: [objects#355] Fix colour preset settings of the Stand-Up Roller Coaster trains.
- Fix: [objects#355] Fix colour preset settings of many vehicles from Wacky Worlds and Time Twister.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Add something new to the game. network version Network version needs updating - double check before merging! park file version Requires updating the park file version number testing required PR needs to be tested before it is merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants