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
Sync autotune-mdi changes back into dev #374
Merged
Merged
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
Dev to autotune-mdi
* New tool to sync profile data from OpenAPS to Nightscout * Hashing the secret * Added tool to package.json * Support both hashed and unhashed api key * Added --preview support. Code run through js-beautify. Less console logging. * Fixed bug with parameter checking order, better logging in case of error / success on upload * Added profile JSON validity checking * One more check for the downloaded profile
It may also make sense to look for !(current.timestamp) && current.insulin since I believe HAPP may give you the ability to edit the "Entered by" string (need to confirm that though).
* script for autotuning based on day of week * +x * exit on usage * accept command-line arguments
* Added additonal logic for HAPP Temp basals Needed to change the timestamp and rate to created_at and absolute respectively. * Update history.js * Update history.js
Conflicts: package.json
Tested this against one of our rigs and confirmed it works. Also added some |
Will merge to dev, probably tomorrow, if no objections. |
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 needs testing to see if it's backwards compatible with running autotune on an OpenAPS rig.