forked from openaps/docs
-
Notifications
You must be signed in to change notification settings - Fork 0
Updating to work on Windows based Edison Setup #1
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
Open
taylorfowler
wants to merge
2,846
commits into
taylorfowler:master
Choose a base branch
from
openaps:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
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
Added one instruction
Fix verbiage about ignored carbs, make urls clickable, remove broken image
Add Enhanced Radio Devices link for antenna
Adding name (making first pull request)
Extended explanation of T-Mobile's brain dead disabling of bluetooth tethering on phones purchased through their stores and additional difficulty of hacked repairs on Samsung S10 series phones. Generic title instead of only for Edison. Might want to update with anything Pi specific.
* Average delta explanation After confusion over what the short_avg_delta and long_avg_delta fields represent to me, an AAPS user. After much confusion over what was meant by the terms someone pointed me to this document which disagrees with what I was told by a developer. * Further clarify short_avgdelta and long_avgdelta The suggested revision was still unclear on how the averaging is performed. I think this should be unambiguous.
Master to dev update
Fix links.
Clarify that any one action listed and not all actions listed are required to turn off the loop. Change "basal type" to "temp basal type".
Fix links and typos. Make some small clarifications.
Add suggestion to protect connections between LiPo battery and its wires.
I think that you meant it without the 's'...
This could also be in the hardware section, but it might be better to be really, really up front about this?
first PR. working on getting everything set up
Added name to loop in progress.
Update loops-in-progress.md
Mention Raspberry Pi Imager as the recommended app to use to install Pi OS to an SD card. (And add ticks around wpa_supplicant.conf filename.)
* Update pi-install.md Until updates made to install file this will help anyone starting a new rig to set up and go! * Clarify Co-authored-by: Scott Leibrand <[email protected]>
On my computer and also on ubuntu ec2 instance the line: "oref0-upload-profile --switch ./myopenaps/autotune/profile.json $NS_SITE $API_SECRET " only works if you change the orderof the arguments to "oref0-upload-profile ./myopenaps/autotune/profile.json $NS_SITE $API_SECRET --switch=true"
Update loops-in-progress.md
Since github deprecated git:// URLs, the openaps-install.sh from the master branch no longer works. Download openaps-install.sh from the dev branch, and install from the dev branch, until we release 0.7.1 to master.
install from dev; add optional while loop to retry flashall.sh
comment for pip install parsedatetime==2.5
rewind, prime and battery indicates change
…eased to master This reverts commit 3444f4f.
Revert "run openaps-install.sh from dev branch" now that 0.7.1 is released to master, also update pump page with recent details.
Completing the first PR as a practice...thanks!
The table was not displaying due to missing start and end pipes.
Fixed table layout
For some reason triple backticks were added around the table, causing it to become a code block instead of a table. I also realized that there were some other inconsistencies in the use of spacing characters, and extra columns on one of the rows. Hopefully this attempt will work better. 🤞
Second try at correcting the table
My third and hopefully last attempt at fixing the table layout. Last try was #1589.
Table update: third time's the charm
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.
No description provided.