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

skip_neutral_temps only if SMB's somehow disabled #33

Closed
wants to merge 1 commit into from

Conversation

mountrcg
Copy link
Contributor

@mountrcg mountrcg commented Dec 9, 2024

This contains an oref fix, which only enables this features if SMB's are disabled (by whatever reason). Because the functionality only makes sense if the MDT pump cannot make any noise anyways and SMB's will always make the pump beep and rattle. So if SMBs can be given there is no reason to skip neutral temps as it would be beeping because of an SMB.

Compare PR with oref PR discussion at openaps/oref0#1452 and issue ticket openaps/oref0#1451

log message for not skipping neutral temps while SMB's are possible
@mountrcg mountrcg changed the title skip_NetralTemps only if SMB's somehow disabled skip_neutral_temps only if SMB's somehow disabled Dec 9, 2024
@mountrcg
Copy link
Contributor Author

replaced by PR #35

@mountrcg mountrcg closed this Dec 11, 2024
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.

1 participant