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

June 27: More preset updates #572

Merged
merged 14 commits into from
Jul 18, 2023

Conversation

hobinjk
Copy link
Collaborator

@hobinjk hobinjk commented Jul 18, 2023

No description provided.

Adds a new split sigil for earth + torment with a sketchy rough
approximation of proc rates
Unrelenting Criticism is now a trait powerful enough to make 8 page FB a
thing of the past. This comments out the template while leaving the
presets just in case.
Funnily enough, all the uptime values one would expect to change were
exactly the same in this log
@hobinjk hobinjk self-assigned this Jul 18, 2023
@marcustyphoon
Copy link
Collaborator

marcustyphoon commented Jul 18, 2023

Our documentation is bad on this—should be a comment in the file—but note that the purpose of the schema dictionary file is that @gw2princeps' URL sharing compression code uses it to translate IDs to numerical indices and back according to the order in these arrays, and thus new entries must go at the end to preserve the index values of all previous entries!

@marcustyphoon marcustyphoon changed the base branch from main to staging July 18, 2023 20:44
@marcustyphoon marcustyphoon merged commit 638206c into discretize:staging Jul 18, 2023
2 checks passed
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.

2 participants