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

refactor/improve monitor mode selection #8804

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

ikalco
Copy link
Contributor

@ikalco ikalco commented Dec 22, 2024

Describe your PR, what does it fix/add?

aims to

  • always ensure a mode is selected, if requested fails still set one, i might add an error notification
  • clarify mode selection code
  • consistent logs

also fixes format selection trying to use INVALID

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

Is it ready for merging, or does it need work?

yes probably
maybe some other people should test too, idk who tho

@ikalco
Copy link
Contributor Author

ikalco commented Dec 22, 2024

this is ready for review
just need figure out how to test custom modes and do that

edit:
highrr is broken cause of sorting stuff
will get to it later
so no review needed yet

@ikalco ikalco force-pushed the improve_monitor_rule_mode_selection branch from 8c14725 to 7030c98 Compare December 23, 2024 02:29
@ikalco
Copy link
Contributor Author

ikalco commented Dec 23, 2024

ready for review

rebased + fixed
tested highrr, highres, requested mode, custom mode, and all work for me ™️
also fallbacks are working

@ikalco ikalco changed the title refactor/improve monitor mode selection (WIP) refactor/improve monitor mode selection Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant