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

Add support for MK3S Revo HT 40W hot end #4487

Closed
wants to merge 4 commits into from
Closed

Conversation

b
Copy link

@b b commented Nov 9, 2023

This code adds support to the MK3S for the e3d Revo high temperature hot end with the 40W precision heater and PT1000 thermistor. Works on my machine.

Copy link

github-actions bot commented Nov 9, 2023

All values in bytes. Δ Delta to base

Target ΔFlash ΔSRAM Used Flash Used SRAM Free Flash Free SRAM
MK3S_MULTILANG -1132 -18 246032 5638 7920 2554
MK3_MULTILANG -1128 -16 245318 5647 8634 2545

@3d-gussner 3d-gussner self-assigned this Nov 10, 2023
@3d-gussner 3d-gussner added the REVO E3D REVO label Nov 10, 2023
@3d-gussner
Copy link
Collaborator

@b Thanks for the PR I forwarded this to E3D as your PR ONLY contains ONE set of data points.

#define NOZZLE_TYPE "E3DREVO_HT_40W"

// Printer name
#define CUSTOM_MENDEL_NAME "Prusa i3 MK3S-RHT40"
Copy link
Collaborator

@3d-gussner 3d-gussner Nov 15, 2023

Choose a reason for hiding this comment

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

Name is to long it is limited to 17 chars at the moment. Also the PR contains a PT1000 thermistor and as there is a HT PT100 version available the naming should also be different.

01234567890123456789
Prusa MK3S-RHT1k          //for E3d Revo HT with PT1000

the other 3 options could be

01234567890123456789
Prusa MK3S-RHT1c          //for E3d Revo HT with PT100 
Prusa MK3S-RUHT1c       //for E3d Revo UHT with PT100
Prusa MK3S-RUHT1k       //for E3d Revo UHT with PT1000

@b
Copy link
Author

b commented Dec 13, 2023

Is there anything else I need to do to make this PR acceptable for merging?

@3d-gussner
Copy link
Collaborator

@b I have been in contact with E3D about this PR and there are few reasons we didn't approved this PR

  1. The max high temperature of 405°C may soften the plastic parts which are made in PETG (ASA/ABS fan-duct). The extruder would need to be printed in a different material that can withstand the heat radiation.
  2. The TM values base only on YOUR printer and will not represent the wider possible range.
    a. For the E3DRevo TM values we collected multiple test results from the community and E3D also provided more data to get "good" average TM values for these.
    b. So you may need to ask other community members to test and update then the TM values
  3. Each variant must have it's own variant and TM file. Please don't use this and this anymore
    a. I know there is the existing #elif defined but this is an very old community merge and we won't change it

Hope that you get some help from the community so we can consider to review this PR again.
The provided the E3D REVO hex files in the official release builds are an exception as E3D approved these.
But please understand that we will not build and provide the hex files for each 3rd party variant there may come in the official releases. Meaning the community will have to compile the firmware themself.

@3d-gussner
Copy link
Collaborator

Closing as no interaction or update

@3d-gussner 3d-gussner closed this Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
REVO E3D REVO
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants