Skip to content
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

PR for dev to master in prep for 0.5.0 release #424

Merged
merged 891 commits into from
Jul 14, 2017
Merged

PR for dev to master in prep for 0.5.0 release #424

merged 891 commits into from
Jul 14, 2017

Conversation

scottleibrand
Copy link
Contributor

No description provided.

Conflicts:
	lib/determine-basal/cob-autosens.js
	lib/profile/index.js
Conflicts:
	lib/determine-basal/determine-basal.js
	lib/profile/index.js
Conflicts:
	bin/oref0-pump-loop.sh
tynbendad and others added 3 commits July 3, 2017 17:27
* Fix color and spacing around token auth in setup script

* change token auth default from Y to N for now, and whitespace
* find last BG from before a meal, and interpolate any gaps

* log autosens ratio in pump-loop.log; better rounding

* consistent y/[N] prompts
@RedCyclist1
Copy link

with SMB/UAM on a Medtronic 723, Dexcom G5,
We had one hiccup in Step 3 to run oref0 set up.

Error: "Please, commit your changes or stash them before you can merge."

Once Scott provided instructions to run "cd ~/src/oref0 and then git stash is likely all you need, and then you can re-run the command that failed."

It made sense. and has run all day with no errors. We are setting up second rig now and did not experience the previous error, it went straight to running script to update to Dev 0.5.0.

Both rigs up and running SMB/UAM

@alternateal
Copy link

G5 / 754WW

Loaded from scratch - get this as result of trying to loop - never seems to try running mmtune.

Starting supermicrobolus pump-loop at Wed Jul 5 20:23:35 BST 2017 with 24 second wait_for_silence:
Waiting up to 4 minutes for new BG: glucose.json newer than pump_loop_completed
Radio ok. Listening: .No pump comms detected from other rigs (this is a good thing!)
mmeowlink.exceptions.CommsException: No acknowledgement from pump on wakeup. Is it out of range or is the battery too low?
Checking pump status (suspended/bolusing): mmeowlink.exceptions.CommsException: No acknowledgement from pump on wakeup. Is it out of range or is the battery too low?
grep: monitor/status.json: No such file or directory
Error, retrying
Sleeping 1;

@scottleibrand
Copy link
Contributor Author

We're working on some improvements for that in #523 if you want to try that.

scottleibrand and others added 3 commits July 5, 2017 16:54
* mmtune 25% of the time; remove unless() loop

* per @Bender1061 suggestion, retry preflight model check

* change 'retrying' language since we don't; remove upto10s sleep

* space

* move unsuspend_if_no_temp stuff into Retrying SMB checks
Fix Echo errors for MDT
@alternateal
Copy link

#523 appears to have worked to kick mmtune in faster.

@alternateal
Copy link

G5. 754WW

oref0 - works
SMB - works
UAM - works

Needed to do a little bit of work to get iPhone BT hotspot to work after installing this version - didn't work as per docs, needed to add a couple of lines to oref0-bluetoothup.sh to force the connection to the hotspot. Not got it to pull the MAC address into the script so I have put my MAC address in the file. This seems to work well and flips back onto wifi when available.

#!/bin/bash

if ! ( hciconfig -a | grep -q "PSCAN" ) ; then
sudo killall bluetoothd
sudo /usr/local/bin/bluetoothd --experimental &
fi

if ( hciconfig -a | grep -q "DOWN" ) ; then
sudo hciconfig hci0 up
sudo /usr/local/bin/bluetoothd --experimental &
sudo bt-pan client XX:YY:ZZ:AA:BB:CC
sudo dhclient bnep0
fi

if !( hciconfig -a | grep -q $HOSTNAME ) ; then
sudo hciconfig hci0 name $HOSTNAME
fi

@scottleibrand
Copy link
Contributor Author

@alternateal oref0-bluetoothup.sh isn't supposed to do the bt-pan client part. That is done in oref0-online: https://github.com/openaps/oref0/blob/dev/bin/oref0-online.sh#L45

* no newline after Continue prompt

* create echocolor-n function for echo -n behavior
* switch non-SMB users over to oref0-pump-loop

* run refresh_old_pumphistory after refresh_old_pumphistory_24h

* use pumphistory-merged.json for ns-meal-carbs

* remove debugging line

* remove debugging line

* wait for  seconds of silence on Old pumphistory-24h

* no need to print out enact/smb-suggested.json
@danamlewis
Copy link
Contributor

Big thanks to everyone who's tested previously; we fixed a couple of issues, and are now needing another round of testing so we can ready this for release! Please do re-pull dev and rerun setup and share your setup so we know we've tested the different options (WW pump, different cgms, with advanced features/without, etc.). Thanks again to everyone who's contributed with testing and/or code for this big release!

@tim2000s
Copy link
Contributor

I've run two setups successfully. One running on xdripaps with iPhone link up for off wifi, and had no issues and the other is an NS only instance that is static at home. Both haven't chucked anything curveball like at me, and I've seen no issue on the dev branch so happy to give a big 👍 On those. RPi3 is a different issue and it's not enjoying the current dev branch at all.

@Bender1061
Copy link
Contributor

Working for me. MDT 723 Edison with Explorer and Edison with TI, using SMB. 👍

@alternateal
Copy link

Ok, new Edison and explorer flashed from scratch. G5 with 754WW - thumbs up from me to go to prod. Bluetooth works as intended also :-)

@Kdisimone
Copy link
Contributor

Worked on a brand new edison with 723 and g4. used auto setup and updated to oref1. all good

* move unsuspend_if_no_temp somewhere it can actually be executed

* move unsuspend_if_no_temp somewhere it can actually be executed

* typo

* move unsuspend_if_no_temp somewhere it can actually be executed

* if smb_verify_enacted, check if suspended
@PieterGit
Copy link
Contributor

Fresh install with 754 and g4 on USB on rpi3. No problems with installing. Found one small issue #543 (don't know if it is a regression).
RPI seem to respond slower than I used too (on the terminal), but load averages are about 1.0.

@dramageek
Copy link
Contributor

Upgrade install (updated the repository, re-ran oref0-setup), explorer board, 722NA.

Works for me both with/without SMB. Bluetooth still works, Nightscout token auth too.

@danamlewis
Copy link
Contributor

Huge thanks everyone for the final round of testing - seeing green lights and turtles all the way down, so will do the release. (Docs PR to match will be openaps/docs#942 if anyone wants to go thumbs that up while we do oref0 0.5.0 release ;))

Conflicts:
	bin/oref0-setup.sh
	package.json
@danamlewis danamlewis merged commit a492c5e into master Jul 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.