-
Notifications
You must be signed in to change notification settings - Fork 180
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 new radio_locale parameter for Medtronic pumps #112
Conversation
What would be an example of the usage for this, so we can test it? |
Related change: openaps/openaps#112
@scottleibrand openaps/oref0#276 uses it. Please let me know if you're happy with this structure. This particular change will modify pump.ini to contain this:
mmeowlink will look for that parameter when doing mmtune etc as per oskarpearson/mmeowlink@3a8d867 (currently only in the dev branch - but having the value in the ini file won't break anything if it's not used) |
Cool. Might be easiest to go ahead and merge this for testing. Does anyone see any reason not to do that? |
Related change: openaps/openaps#112
I think that this can be merged. We could need to make additional changes, but from what I can tell it'll work as expected. |
Related change: openaps/openaps#112
Per https://gitter.im/nightscout/intend-to-bolus?at=5858a0f6c5a4e0233ba7961c it doesn't appear that |
@scottleibrand and @oskarpearson : i think pull request needs still to be applied to an openaps release to be able to remove the workaround in the current
|
Add support for the 'radio_locale' medtronic parameter.
This helps make using oskarpearson/mmeowlink@3a8d867 easier.