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

undefined variable dev5m in lib/autotune-prep/categorize.js dev branch #1417

Closed
Zanfib opened this issue Dec 21, 2021 · 2 comments
Closed

undefined variable dev5m in lib/autotune-prep/categorize.js dev branch #1417

Zanfib opened this issue Dec 21, 2021 · 2 comments

Comments

@Zanfib
Copy link

Zanfib commented Dec 21, 2021

I all,
I'm new to openaps and try to have an open loop running.
I've cloned the git dev branch of oref0 to use autotune on my linux desktop and faced an error with running autotune :

/home/didier/Developpement/oref0/lib/autotune-prep/categorize.js:224
dev5m = delta-BGI;
^

ReferenceError: dev5m is not defined
at categorizeBGDatums (/home/didier/Developpement/oref0/lib/autotune-prep/categorize.js:224:15)
at generate (/home/didier/Developpement/oref0/lib/autotune-prep/index.js:23:30)
at Object. (/home/didier/Developpement/oref0/bin/oref0-autotune-prep.js:135:27)
at Module._compile (internal/modules/cjs/loader.js:999:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)
at internal/main/run_main_module.js:17:47

I've simply added 'var dev5m;' in the file lib/autotune-prep/categorize.js line 153 and it worked as expected.
Thanks !

@scottleibrand
Copy link
Contributor

Thx. I opened #1417 to fix.

scottleibrand added a commit that referenced this issue Jan 5, 2022
* comment out shared-node cp to avoid full disk

* #1417: define undefined var dev5m
bjornoleh pushed a commit to bjornoleh/oref0 that referenced this issue Jan 9, 2022
* comment out shared-node cp to avoid full disk

* openaps#1417: define undefined var dev5m
bjornoleh pushed a commit to bjornoleh/oref0 that referenced this issue Jan 12, 2022
* comment out shared-node cp to avoid full disk

* openaps#1417: define undefined var dev5m
@ChanceHarrison
Copy link

I believe this issue can be closed now that #1418 has merged.

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

No branches or pull requests

3 participants