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

SC2 WoL: Mod, Item and Location update #2113

Merged
merged 14 commits into from
Sep 15, 2023

Conversation

Ziktofel
Copy link
Collaborator

@Ziktofel Ziktofel commented Aug 20, 2023

What is this fixing or adding?

Migrates SC2 WoL world to the new mod with new items and locations. The new mod has a different architecture making it more future proof (with planned adding of other campaigns). Also gets rid of several old bugs

Adds new short game formats intended for sync games (Tiny Grid, Mini Gauntlet). The final mission isn't decided by campaign length anymore but it's configurable instead. Allow excluding missions for Vanilla Shuffled, corrected some documentation.

NOTE: This is a squashed commit with Salz' HotS excluded (not ready for the release and I plan multi-campaign instead)

How was this tested?

Completed several games, including multiworld async, Joining release games works (but there are some differences in UX given the way how the new mod works)

Also generated and verified a lot of games if they got generated correctly

If this makes graphical changes, please attach screenshots.

The tracker got updated for the new stuff

Snímek obrazovky pořízený 2023-08-13 00-32-01
Snímek obrazovky pořízený 2023-08-08 19-33-01
Snímek obrazovky pořízený 2023-08-08 19-32-46

@KonoTyran
Copy link
Contributor

I'm not entirely sure how big this is, as I can't even load the full files changed tab, but having so many misc files for the mod in the main AP repo is a no go.
From what I could load it looked like every sound file used in the WOL campaign is inside this PR. from what I understand the licenses of Starcraft allow this, (which is weird, but cool I guess).

I would highly suggest you to host the mod in your own repo, and have the python client that is shipped with AP just download the needed mod files from that repo. Any files included here will have to be downloaded by all people wishing to play AP, as they all get included in the installer. so we really don't want people to have to download the entirety of WoL inside of the AP installer if they have no intention of playing StarCraft in AP.

@PoryGone
Copy link
Collaborator

Since it was requested I review this here, I'll repeat here what I said on Discord:

This PR is absolutely absurd. I don't want to have to clone all of this crap, and I'm of the opinion that it should be immediately rejected on principle.

@Ziktofel
Copy link
Collaborator Author

Ziktofel commented Aug 25, 2023

The other thing that the mod version shall be synchronized with the client version

See also https://discord.com/channels/731205301247803413/731214280439103580/1144607092112625754

@Ziktofel
Copy link
Collaborator Author

I've moved the map/mod data to a new repo: https://github.com/Ziktofel/Archipelago-SC2-data/tree/sc2-wol-new

@Berserker66 Berserker66 merged commit 648d682 into ArchipelagoMW:main Sep 15, 2023
14 checks passed
@ThePhar ThePhar changed the title SC2 WoL - Mod, Item and Location update SC2 WoL: Mod, Item and Location update Oct 16, 2023
@ThePhar ThePhar added is: documentation Improvements or additions to documentation. is: enhancement Issues requesting new features or pull requests implementing new features. affects: webhost Issues/PRs that touch webhost and may need additional validation. labels Oct 16, 2023
@Ziktofel Ziktofel deleted the sc2-wol-new branch October 27, 2023 23:33
FlySniper pushed a commit to FlySniper/Archipelago that referenced this pull request Nov 14, 2023
Migrates SC2 WoL world to the new mod with new items and locations. The new mod has a different architecture making it more future proof (with planned adding of other campaigns). Also gets rid of several old bugs

Adds new short game formats intended for sync games (Tiny Grid, Mini Gauntlet). The final mission isn't decided by campaign length anymore but it's configurable instead. Allow excluding missions for Vanilla Shuffled, corrected some documentation.

NOTE: This is a squashed commit with Salz' HotS excluded (not ready for the release and I plan multi-campaign instead)

---------

Co-authored-by: Matthew <matthew.marinets@gmail.com>
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
Migrates SC2 WoL world to the new mod with new items and locations. The new mod has a different architecture making it more future proof (with planned adding of other campaigns). Also gets rid of several old bugs

Adds new short game formats intended for sync games (Tiny Grid, Mini Gauntlet). The final mission isn't decided by campaign length anymore but it's configurable instead. Allow excluding missions for Vanilla Shuffled, corrected some documentation.

NOTE: This is a squashed commit with Salz' HotS excluded (not ready for the release and I plan multi-campaign instead)

---------

Co-authored-by: Matthew <matthew.marinets@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects: webhost Issues/PRs that touch webhost and may need additional validation. is: documentation Improvements or additions to documentation. is: enhancement Issues requesting new features or pull requests implementing new features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants