luc-github / ESP3D-WEBUI

A Web UI for ESP8266 or ESP32 based boards connected to 3D printers / CNC
GNU General Public License v3.0
762 stars 305 forks source link

Added support for Marlin T0 redundant, probe and chamber temperatures #103

Closed BToersche closed 4 years ago

BToersche commented 4 years ago

Changes in the Temperatures tab. The redundant and probe temperatures do not have controls associated with them, since they cannot be controlled after all. Each of the temperatures have been added to the 2 graphs at the bottom:

Added options for enabling/disabling to the preferences menu (defaults are: "disabled"). image

Some notes:

luc-github commented 4 years ago

interresting I did not know the M105 R it is not documented in GCODE list : https://marlinfw.org/docs/gcode/M105.html

same for Probe and Chamber

are these controls compatible in repetier and smoothieware ? I did not see any fw check

BToersche commented 4 years ago

Rather than waiting for my probe to arrive, I've decided to test the code by editing the pins in my printer's firmware such that the probe and chamber temps would read a copy of the already installed probes. All works as it should: image

In terms of firmware compatibility, I've checked each of the firmwares to see which of the thermistors are supported:

Marlin

Marlin Kimbra

Repetier (Both ArduinoAVR and ArduinoDUE)

(*) Repertier is able to show chamber temperatures, however these are stub temperatures copied from extruder0.

Repetier for Davinci (Both ArduinoAVR and ArduinoDUE)

Grbl

Smoothieware

(*) By default, the configuration files only include hotend and bed temperature_control, however it seems you can simply add the required temperature_control modules, since you can change the designator (R/P/C, line 140), the GCode (to M141 for chamber, line 135) and not set a heater_pin for readonly mode (for redundant and probe, line 162). Though, this possibility might not immediately be clear for users.

For Smoothieware it might be a good idea to add some information next to the options, because people might not know the required designator for each of the thermistors. Of course this information is hidden when another firmware is used: image Let me know if you agree.

I'll add the required firmware checks to the PR tomorrow or the day after.

luc-github commented 4 years ago

No need to add smoothieware information in webUI should be ok to be in wiki

So please hide when not applicable in preferences - UI should follow also there are marlin_embeded which act as marlin and grbl_embeded which act as grbl

I will see how to add these in 3.0 3.0 is more flexible than 2.X- it build UI according the content of M105 answer, and because in 3.0, the poilling command is configurable by user, no need to have new preferences setting to choose M105 vs M105 R

luc-github commented 4 years ago

Also looking at UI screenshot - redondant should not have target output if cannot be controled, or I missed something? I guess probe should be same

BToersche commented 4 years ago

I've added the required firmware checks for the temperature controls and tested these on my machine. Like stated, my machine runs Marlin, so I can only really test whether or not it works for the Marlin firmware. I did however also test to see if the options would be properly hidden if I removed Marlin from the supported firmwares, which worked correctly. image

As for the target temperature values: When a target is set for Heater T0, Marlin will also report this target temperature for the redundant sensor. This makes sense considering the redundant sensor is measuring the same heater as T0: image As for the Probe, I intentionally kept the target temperature value in the UI as well because:

Also: I merged the base branch into this branch and rebuild the index.html.gz files including your changes.

luc-github commented 4 years ago

Ok thank you for clarification - I will review on my side and merge it once done

luc-github commented 4 years ago

I have added same support in 3.0 image

luc-github commented 1 year ago

@all-contributors please add @BToersche code improvement

allcontributors[bot] commented 1 year ago

@luc-github

I've put up a pull request to add @BToersche! :tada: