Add DeepStats core functionality without external backend#44
Draft
jmichelsen wants to merge 8 commits intoblizzhackers:mainlinefrom
Draft
Add DeepStats core functionality without external backend#44jmichelsen wants to merge 8 commits intoblizzhackers:mainlinefrom
jmichelsen wants to merge 8 commits intoblizzhackers:mainlinefrom
Conversation
This is a skeleton feature. It does not have any backend attached so you must build your own backend to recieve the data
This will check your chars skill allocation and estimate the build you are running
Contributor
Author
|
You guys are of course under no obligation to merge this, but I wanted it available in case it's useful to anyone. |
Contributor
|
I think this is good, agree it shouldn't point to any default server |
Fa-b
reviewed
May 5, 2020
| experience: 0, | ||
| deaths: 0, | ||
| lastArea: "", | ||
| lastAreaID: 0, |
Contributor
There was a problem hiding this comment.
Adding lastAreaID will then also appear in current logging which should be fine but I think you wanted to avoid that.
You could also maybe use sdk.js instead to parse the ID from string.
| DeepStats.create(); | ||
| } | ||
|
|
||
| string = Misc.fileAction("data/deepstats/" + me.profile + ".dump.json", 0); |
Contributor
There was a problem hiding this comment.
A bit worried about the potential file size.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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 has the scaffolding and some example code to make POST requests from kolbot when items drop, as well as when SOJs are sold or Diablo walks.
For DeepStat file writes, a sample of the output is included.
In the stats, every item is given a unique ID. It follows the same format as the unique ID given by mulelogger, except it has a timestamp prepended to it: