-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add time_buckets to EarningsTracker data #228
Closed
ksedgwic
wants to merge
8
commits into
ZmnSCPxj:master
from
ksedgwic:2024-08-earnings-tracker-time-buckets
Closed
Add time_buckets to EarningsTracker data #228
ksedgwic
wants to merge
8
commits into
ZmnSCPxj:master
from
ksedgwic:2024-08-earnings-tracker-time-buckets
Commits on Aug 12, 2024
-
factor parse_json into Jsmn::Object static method
This makes it easier to generate test cases by using literal JSON.
Configuration menu - View commit details
-
Copy full SHA for b96c743 - Browse repository at this point
Copy the full SHA b96c743View commit details -
This makes it easier to write test cases by comparing to expected values
Configuration menu - View commit details
-
Copy full SHA for 46f9dda - Browse repository at this point
Copy the full SHA 46f9ddaView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6188ffa - Browse repository at this point
Copy the full SHA 6188ffaView commit details -
add get_now() and mock_get_now() to EarningsTracker and test_earnings…
…tracker A time source is needed for upcoming time buckets change.
Configuration menu - View commit details
-
Copy full SHA for 721a4f0 - Browse repository at this point
Copy the full SHA 721a4f0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 528fcd9 - Browse repository at this point
Copy the full SHA 528fcd9View commit details -
Upgrade EarningsTracker to time bucket schema, use old semantics
This commit modifies the schema of EarningsTracker to allow storing and accessing earning and expenditure data in specific time ranges. All existing strategies and reports still use all data from all time so this PR should not change any balancing behavior. After we've run w/ this for a while we'll have time-based data collected and can evaluate how to improve the strategies.
Configuration menu - View commit details
-
Copy full SHA for 5bc9075 - Browse repository at this point
Copy the full SHA 5bc9075View commit details -
Configuration menu - View commit details
-
Copy full SHA for 55479bb - Browse repository at this point
Copy the full SHA 55479bbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 13a4e8f - Browse repository at this point
Copy the full SHA 13a4e8fView commit details
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.