Skip to content

Add initial version of compatiblity scenario text objects#350

Merged
Gymnasiast merged 8 commits intomasterfrom
scenario_texts
Oct 30, 2024
Merged

Add initial version of compatiblity scenario text objects#350
Gymnasiast merged 8 commits intomasterfrom
scenario_texts

Conversation

@AaronVanGeffen
Copy link
Member

@AaronVanGeffen AaronVanGeffen commented Oct 27, 2024

Dumped using a script: readParkStrings.py

Copy link
Member

@tupaschoal tupaschoal left a comment

Choose a reason for hiding this comment

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

I love this idea of finally getting rid of that weird piece of text on the strings file.

Not sure how we can help to review, it's certainly too much to manually review. I don't love the indentation used, but then the automatic JSON formatters are usually pretty bad, so no big deal.

@Gymnasiast
Copy link
Member

ZehMatt has written one specifically for this repo. I can run it on the files once they have been generated.

@Gymnasiast
Copy link
Member

Gymnasiast commented Oct 30, 2024

Note to self:

  • There are 163 added files in this PR
  • There are 137 STEX files in RCTC
  • A difference of 26
  • 18 of these are UCES scenarios, leaving 8
  • Tycoon Park and Panda World are present in RCTC, but have no STEX, leaving 6
  • Mega Park, Urban Jungle, Fort Anachronism, Alton Towers, Heide-Park and BPB are not present in RCTC, leaving 0.

@Gymnasiast Gymnasiast marked this pull request as ready for review October 30, 2024 15:54
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.

3 participants