Skip to content

Conversation

@jpcunningh
Copy link
Contributor

G4-Go script is much better than the old bin/oref0-dexusb-cgm-loop.py script.

@jpcunningh
Copy link
Contributor Author

jpcunningh commented Aug 19, 2018

@PieterGit, does the PR accomplish your suggestion in #1085 ?

@PieterGit
Copy link
Contributor

yes, looks good. i'll test it as well.

Copy link
Contributor

@PieterGit PieterGit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, tested install on Edison Explorer Board and works. @scottleibrand I think this can be merged to dev for the 0.7.0 release

@PieterGit
Copy link
Contributor

PieterGit commented Aug 19, 2018

Some history/background. This script from created and used in the < 0.6.0 days (which used quite some more Python than nowadays). This used to work similar to the G4-upload script. The G4-Go CGM is much better, so this should not be used with 0.7.0 anymore and this script can be removed.

@scottleibrand scottleibrand merged commit eaba7ec into openaps:dev Aug 20, 2018
@jpcunningh jpcunningh deleted the remove-unused-oref0-dexusb-cgm-loop branch August 20, 2018 01:57
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.

3 participants