-
Notifications
You must be signed in to change notification settings - Fork 47
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
Easier support for non-USA (WW) pumps #51
Comments
I started doing some work on this at https://github.com/oskarpearson/mmeowlink/tree/radio_parameters - but it needs a lot of work, since it predates many other changes. Structurally, though, I like the way we pass/set parameters. I'll organise to do a rebuild of that tree ASAP |
Related, and which should probably be fixed as part of this change: #16 |
@oskarpearson : can you tell me what needs to be done, and how you think it's the best way to implement it? I can see if I can help a bit but I don't know what changes you intend to make. |
Anyone have any updates on this? Wondering if we should go ahead and merge openaps/oref0#232, or if there is a reasonable path forward toward doing this right... |
If anyone wants to give this a test on a new box, this will be fixed with: openaps/openaps#112 I'll be doing an end-to-end test shortly. |
I've merged the latest changes into master, and built/published a release in preparation of changes to oref0/openaps: mmeowlink 0.10.0 I'm going to leave this open for now - until any issues with those are resolved. |
@oskarpearson i integrated run subg-ww-radio-parameters script on mmtune I still had a problem with the radio_locale, see #55 |
Currently it's not easy to use non-USA pumps with Mmeowlink and OpenAPS.
At the moment, I do this manually in my loop, by running
mmtune.py --port ${PORT} --serial FIXME --radio_locale WW
outside of OpenAPS. Some hacks have been proposed to deal with this, inclduing openaps/oref0#226 (comment)Additionally, some WW pumps require this process, since the default subg radio settings may need some adjustment: https://github.com/oskarpearson/mmeowlink/wiki/Non-USA-pump-settings
It would be better if:
The text was updated successfully, but these errors were encountered: