Skip to content

Conversation

@TheSin-
Copy link

@TheSin- TheSin- commented Dec 7, 2025

  • Add tools if you have more set in the printer profile
  • Add a few variants for chamber like cavity (jneilii firmware uses it)
  • Add TMC2240 temperatures, since most TMCs do not support temp, limit to list (temps do not show unless the driver is energizes, so add or 0 since it will return None when it's not energized. I'm not sure what others drivers should be added so I only added 2240, but it's easy to add to it.

TODO:

  • Check that on FIRMWARE_RESTART or RESTART it rereads and rebuilds as the sensor could have changed, haven't tested this yet to make sure. The both lists might need to be reset when system firmware is triggered?
  • I'd love to add the TMC temps to the temperature pane (doesn't need to be on the graph, but I could not figure out how to append to the temperature container to add and update the temps)
  • Maybe settings could be setup to select temperatures from the web socket list, incase chamber has an other name, and if you wanted to track others like the TMC temps. in Klipper all the temperature books can have an options gcode_id parameter, this indicated to KlipperScreen and other UIs to graph/display that temp, maybe we can find a way to only get those, sadly that value wasn't present in the object query.

FUTURE PRs:

  • removing folder does remove locally, but doesn't refresh (there are errors in the log file) (moonraker needs to be restarted for the refresh to work)
  • adding folder does add locally, but doesn't refresh (there are errors in the log file) (moonraker needs to be restarted for the refresh to work)
  • the root folder is hardcoded to gcodes but this value can be changed in the klipper config, we might want to see if we can get the value, I haven't checked the moon raker api to see if it's fetch able yet, but wanted to log it.

- Add tools if you have more set in the printer profile
- Add a few variants for chamber like cavity (jneilii firmware uses it)
- Add TMC2240 temperatures, since most TMCs do not support temp, limit to list (temps do not show unless the driver is energizes, so add `or 0` since it will return None when it's not energized.  I'm not sure what others drivers should be added so I only added 2240, but it's easy to add to it.
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