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

reduce bus/log spam #37

Merged
merged 4 commits into from
Apr 14, 2024
Merged

reduce bus/log spam #37

merged 4 commits into from
Apr 14, 2024

Conversation

JarbasAl
Copy link
Member

@JarbasAl JarbasAl commented Apr 12, 2024

  • rename methods for consistency (show/display/update_gui ... what a mess)

  • make the scheduled event update only mk1 faceplate, no need to spam the GUI bus every 10 seconds (this isnt the homescreen clock)

  • remove references to deprecated self.enclosure.display_manager

  • add settings changed callback, only send the repeating events if the setting is enabled, no need to spam debug logs otherwise!

  • rm create_skill deprecated function

  • drop .qml file extension from pages, preparing for alternative GUIs support

  • move ui folder to qt5 , further silencing logs

  • remove dead files and outdated automations (skill.json)

- rename methods for consistency (show/display/update_gui ... what a mess)

- make the scheduled event update only mk1 faceplate, no need to spam the GUI bus every 10 seconds (this isnt the homescreen clock)

- remove references to deprecated self.enclosure.display_manager

- add settings changed callback, only send the repeating events if the setting is enabled, no need to spam debug logs otherwise!

- rm create_skill deprecated function
@JarbasAl JarbasAl requested review from goldyfruit, emphasize, mikejgray and a team April 12, 2024 17:25
@JarbasAl JarbasAl added the bug Something isn't working label Apr 12, 2024
@JarbasAl JarbasAl changed the title modernize reduce bus/log spam Apr 12, 2024
rm dead files and bad automations
Copy link
Member

@builderjer builderjer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works with my Mark1 just fine. Awesome!!

@JarbasAl JarbasAl merged commit 5cd270a into dev Apr 14, 2024
2 of 8 checks passed
@JarbasAl JarbasAl deleted the housekeeping branch April 14, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants