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

TUNIC: Implement New Game #2172

Merged
merged 108 commits into from
Jan 12, 2024

Conversation

silent-destroyer
Copy link
Contributor

What is this fixing or adding?

Adds Archipelago support for the game TUNIC.

The client mod can be found here: https://github.com/silent-destroyer/tunic-randomizer-archipelago

How was this tested?

Several test sessions among members of the community, and continuous updates to the apworld and client based on feedback/bug reports.

If this makes graphical changes, please attach screenshots.

image
image

silent-destroyer and others added 30 commits July 29, 2023 12:30
…essage, but importantly it does generate now
Some fixes to get it to generate, and some little formatting things
CSV usage fix for apworld compatibility
Location and Item name grouping
Some changes to docs, some might be a little nitpicky
Added more location groups, and a default group as the region name
@ThePhar ThePhar added the is: new game Pull requests for implementing new games into Archipelago. label Oct 16, 2023
Copy link
Collaborator

@el-u el-u left a comment

Choose a reason for hiding this comment

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

Gonna give this one a checkmark with the following caveats:

  • I never played this game; just looked at the code
  • Test coverage is sparse
  • World code does not fully conform to AP code style (many lines go over 120)

worlds/tunic/__init__.py Outdated Show resolved Hide resolved
@silent-destroyer
Copy link
Contributor Author

Merging the entrance rando feature here now so that it can be reviewed and included in the initial tunic merge.

silent-destroyer and others added 9 commits December 10, 2023 11:31
* Update en_Tunic.md
* ER now uses events rather than a series of can_reach's. Slightly slower gen, better spoiler output.
* Adds an option to specifically place Laurels at either 6 coins, 10 coins, or 10 fairies.
* Logic options - Restricted, NMG, Unrestricted
* Maskless option (so that NMG and Unrestricted don't specifically imply maskless)
* Lanternless option (so that NMG and Unrestricted don't specifically imply lanternless)
@Berserker66 Berserker66 merged commit 3933fd3 into ArchipelagoMW:main Jan 12, 2024
12 checks passed
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
@silent-destroyer silent-destroyer deleted the tunic-randomizer branch July 26, 2024 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: new game Pull requests for implementing new games into Archipelago.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants