-
Notifications
You must be signed in to change notification settings - Fork 11
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
Version 3.0.0 #28
Closed
Closed
Version 3.0.0 #28
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Also add documentation for getting this building locally.
This sets the stage for randomizing to all available locations, including shops.
This allows us to give multiple items in a much better way.
This still needs to be integrated into the online logic.
This doesn't use minhook from vcpkg because that version seems to be incompatible somehow, and we want to start using the ModEngine2 hook API anyway.
nex3
force-pushed
the
full-rando-server
branch
from
August 20, 2024 07:53
a3bd362
to
b326078
Compare
Turns out this downloads with the original filename, not the artifact name! Weird.
nex3
force-pushed
the
full-rando-server
branch
from
August 20, 2024 08:17
e29ea71
to
a326c21
Compare
This doesn't actually run on Proton, which was the primary motivation for providing it, so it's just a waste of resources and confusing for users.
Improve build instructions
This seems to have stopped compiling, so this relies on a fork of the now-defunct ME2 to avoid loading it.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fully implements the changes discussed in #17, integrating this client with a branch of @thefifthmatt's static randomizer so that all item locations can be randomized rather than just unique items.
Major functionality changes:
The static randomizer now replaces all in-world item pickups with synthetic items whose paramdata encodes the Archipelago location ID and (for local items) the DS3 item ID. Specifically:
Uses ModEngine2 rather than manual dinput8.dll injection to be compatible with other mods (particularly the static randomizer). This also makes it compatible with DS3 1.15.2.
More gracefully handles connection and disconnection:
Drops progressive item logic since that's no longer necessary to handle non-unique items.
Supports granting Path of the Dragon through a custom event created by the static randomizer.
Major infrastructure changes:
spdlog
for compatibility with ModEngine2.Instructions for building this client are in the README. You can test it by taking the latest beta release and overwriting its
archipelago.dll
with the one you built locally.Closes #17