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

cgm-loop.log: Not an openaps environment, run: openaps init in papertrail logs; similar in normal logs #1149

Open
mhaeberli opened this issue Nov 24, 2018 · 3 comments

Comments

Projects
None yet
3 participants
@mhaeberli
Copy link
Contributor

commented Nov 24, 2018

Describe the bug
A clear and concise description of what the bug is.

After installing dev branch on a freshly installed Edison, at least in the nightscout/G5 configuration, repeated error messages are seen in the logs, of the form:

(in papertrail, for example):
cgm-loop.log: Not an openaps environment, run: openaps init

Looping seems still to be proceeding normally.

To Reproduce
Steps to reproduce the behavior:

  1. Flash a fresh Edison

  2. Follow normal installation procedures, but hit ctrl-C at the point where it offers to finish installing 'master' oref0

  3. Reboot Edison, set new password as requested

  4. Install dev branch per instructions

  5. Configure and run dev branch via standard oref0-setup interactive setup

  6. Problem can be 'fixed' by adding:
    "cgm_loop_path": "/root/myopenaps/", as the first entry in preferences.json

Expected behavior
I would expect the above needed cgm_loop_path parameter in preferences.json to be set correctly, or the system otherwise to avoid reporting a (spurious) error.

Smartphone (please complete the following information):
Not Applicable, but iPhone 7 with latest production iOS

Setup Information (please complete the following information):

  • Medtronic 723, v.2.4A firmware
  • CGM type (e.g. Dexcom G6 via Dexcom Share and Nightscout)
  • Edison/Explorer Board rig)
  • oref0 dev latest commit hash - 7f3526e

Additional context
N/A

@mhaeberli

This comment has been minimized.

Copy link
Contributor Author

commented Nov 24, 2018

PS props to @cluckj for guidance !

@cluckj

This comment has been minimized.

Copy link
Contributor

commented Feb 25, 2019

Found the bug: oref0-setup.sh does not set cgm_loop_path for "G5" cgm type.

cluckj added a commit to cluckj/oref0 that referenced this issue Feb 25, 2019

jaylagorio added a commit to jaylagorio/oref0 that referenced this issue Mar 2, 2019

Fix connected receivers in 0.7.0-dev, support for G5/G6 touchscreen r…
…eceivers

Changed files:
oref0-setup
- Includes cgm_loop_path fix found by @chuckj for both G5 and G6 (openaps@8799a13 to fix openaps#1149) and includes support for G5/G6 touch receivers (previously under openaps#1182)

oref0-cron-every-minute
- Reincluding support for offline CGM, with optional upload, for G5 and G6 receivers

jaylagorio added a commit to jaylagorio/oref0 that referenced this issue Mar 2, 2019

Add support for touchscreen receivers, fix CGM loop bug for connected…
… receivers

Changed files:
`oref0-setup.sh`
- Includes cgm_loop_path fix found by @chuckj for both G5 and G6 (openaps@8799a13 to fix openaps#1149) and includes support for G5/G6 touch receivers (previously under openaps#1182)

`oref0-cron-every-minute.sh`
- Reincluding support for offline CGM, with optional upload, for G5 and G6 receivers
@jaylagorio

This comment has been minimized.

Copy link
Contributor

commented Mar 2, 2019

Fix included in #1217

scottleibrand added a commit that referenced this issue Mar 10, 2019

scottleibrand added a commit that referenced this issue Mar 11, 2019

Add support for touchscreen receivers, fix CGM loop bug for connected…
… receivers (#1217)

* Add support for touchscreen receivers, fix CGM loop bug for connected receivers

Changed files:
`oref0-setup.sh`
- Includes cgm_loop_path fix found by @chuckj for both G5 and G6 (8799a13 to fix #1149) and includes support for G5/G6 touch receivers (previously under #1182)

`oref0-cron-every-minute.sh`
- Reincluding support for offline CGM, with optional upload, for G5 and G6 receivers

* Readding accidentally removed logging

Reenables apache-chainsaw support

* Resolving conflicts with newly merged changes

* Touchup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.