Update/fix world-heatmap #7227
Open
Update/fix world-heatmap #7227
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
- Added a super dupers important fix that needs to go through before Leagues V is over in order to work (separating Leagues data from regular account data) - Different gamemodes now have their own heatmaps - Automatic fix for file naming scheme - Implemented 3D tracking of data instead of assuming it's all on plane 0 - Data upload time is now determined by age of oldest heatmap instead of individually - Refactored a lot of code - Legacy save file conversion now updates metadata
Fixed bad use of @Inject
Fixed a yuge bug where creating a new config profile and then switching it would result in all if not most heatmaps getting reset and then saved over lul
- Leagues-decontamination is now performed if the user is on the Leagues hiscores (turns out to be a much better way of determining if they ever played leagues than detecting if they ever log onto Leagues in the very few days between the release of V1.6 and the end of Leagues) - Parallelized the file-moving during leagues decontamination
GrandTheftWalrus
changed the title
Added final upload to Leagues-decontamination process
Added date modified reset to Leagues decontamination
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.
It's a pretty huge diff due to the refactoring. I'll sling the reviewer $50 CAD if they're able to review it by Wednesday
Update: It turns out there was a way to fix users' data without the update approval being time-sensitive (and it's much more comprehensive anyway), but I'll still honor the 50 zucks for Wednesday thing