-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
🚧 WIP: Steel sheet check #4832
base: MK3
Are you sure you want to change the base?
🚧 WIP: Steel sheet check #4832
Conversation
Fix rebase issues
All values in bytes. Δ Delta to base
|
#: ../../Firmware/Marlin_main.cpp:5844 ../../Firmware/messages.cpp:163 | ||
#: ../../Firmware/util.cpp:454 ../../Firmware/util.cpp:455 | ||
msgid "Check selected steel sheet." | ||
msgstr "Skontrolujte vybraný platňa." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be "Skontrolujte vybranú platňu"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, will wait for other reviews and commit later
@3d-gussner RO looks good from my perspective |
@3d-gussner NL looks good to me |
French gets a green light from my side. 👍 |
Hungarian should be "Ellenőrizd a kiválasztott az tárgyasztal lapot" |
Thanks for the suggestion. I searched for existing terms and words and for steel sheet I found these: "Please place steel sheet on heatbed." "Please remove steel sheet from heatbed." Can you please also review the rest of the translations (of course so they do not exceed the limits) for "check"? Now I see "Ellenorizd" and "Nezd meg a" (look at ???) are used for "Check" Also you can use the diacritics. In this case "Ellenorizd" should be "Ellenőrizd" or not? |
@3d-gussner For ro I would do |
Thanks I will wait for potential other changes before committing. |
Totally agree, thx !
Le mer. 8 janv. 2025 à 17:07, Ilovemyhouse ***@***.***> a
écrit :
… French gets a green light from my side. 👍
—
Reply to this email directly, view it on GitHub
<#4832 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABCUIHLKSLSE3EYHEMDQ7P32JVEL3AVCNFSM6AAAAABU2EFJVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZYGA2TGNZZGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
"Kérlek, helyzed a lapot a tárgyasztalra." - the literal translation is please place sheet on the object table. In Hungarian we don't really say that it's from steel. (kind of like in the Czech original, if I recall correctly) However, just for consistency sake: Yes, Nézd meg literally means Look at, but it can mean Check as well. I might have used it instead of Ellenőrizd to save two characters. I'm thinking of going through the entire Hungarian translation in the near future to make it more consistent (and to add diacritics) As for the diacritics, Ellenőrizd is correct. |
Please tag me when you start working on that as we may have to make a few changes to the diacritics scripts (enable/disable individual characters from the font based on usage). |
Add Sheet warning bit check Improve `M115` steel check when `Always` is selected for host connection Add some debug output
@AttilaSVK Thanks for the response. Please also tag @leptun when you are done, so we can try to get all used Hungarian diacritics shown correctly. |
after boot.
Thanks @jfestrada for the idea in #4569.
This is a different approach for this request.
We have the https://help.prusa3d.com/materials (which is missing the PP powder coated steel sheet) and we could use it to warn users when they start to print:
Add new parameter Type to the
M850
gcode andHW Setup -> Sheets
to set the type of sheetSmooth
Textured
Satin
NylonPA
PP
Custom
Reuse and add eeprom values needed
previous values are
>10
as the MINtemp for Bed would not allow any lower values so resetting these with255
or default values0-6
is not an issueAlso add
M862.7
to check the compatible sheets with parameterP
. In the PrusaSlicer Filament profiles we could add it to the custom G-codeAs the materials help article shows compatibility ✔️ and warnings ❕and not to use ❌ I would suggest a bit based value
bit 0 = Smooth compatible
bit 1 = Textured compatible
bit 2 = Satin compatible
bit 3 = NylonPA compatible
bit 4 = PP compatible
bit 5 = Custom sheet compatible (like G10 sheets see https://www.youtube.com/watch?v=g0PK4oXbJT8)
Add an additional Check state
Always
orWait
which is the same asWarn
but doesn't timeout and HAS to be confirmed on LCDAlways
anyM115
Gcode will trigger the LCD message to be confirmed.Of course the default setting for the Sheet type check would be warn (with timeout)
ToDos:
Bit check
M862.7 P
And an additional
M862.7
parameterWarn
with same bit based valueIn case there is a warning or stop print send also an
//action:notification
to inform host users.In PrusaConnect we already have some sheet information
Another check we could add is in the Preheat menu, so users which use that will be warned before the starting a print or 1st layer calibration.
Update Materials help page as PP sheet is missing
Optional Add prusa.io/materials link we could use in the warnings
Update PrusaSlicer filament profiles for MK2.5/S and MK3/S/+ printers.
Review translations for
Check selected steel sheet.
Zkontrolujte tiskový plát.
@DRracerÜberprüfe ausgewähltes Stahlblech.
@3d-gussnerVerifique la lámina de acero seleccionada.
Vérifiez la plaque en acier sélectionnée.
@ErwanAliasr1 @IlovemyhousProvjerite odabrani čelični ploca.
Nezd meg a kiválasztott az acellapot.
@AttilaSVK @HauzmanControllare la piastra d'acciaio selezionata.
@wavexxControleer de geselecteerde staalplaat.
@vintagepc @3d-gussner @stelgenhofSjekk valgt stålplaten.
Sprawdź wybraną płyta stalowa.
@arekmVerificați suprafața de print selectată
@leptun @HauzmanSkontrolujte vybranú platňu.
@ingbrzy @shatter136Kontrollera vald metallskivan.
Post release tasks:
Test PR:
following lines
and before
This will add automatically during slicing the needed
M862.7 P<value> W<value>
to the start gcode.I tried to add the
M862.7
to the PrusaSlicer Filament section, but franklyCustom G-Code
s are added after the Printer tab Start G-Code. These kind of checks need to be at the beginning, like the Printer Type Nozzle size checks already are!In LCD Settings -> HW Setup -> Checks -> Sheet default is
use glue stick
In LCD Settings -> HW Setup -> Steel Sheets -> you find a new menu
Type
Ideally you have at lest two Steel sheets calibrated.
Test 1
a. ideally enable some logging
Smooth
sheetM862.7 P7 W0
to simulate a PLA print, which can be printed on Smooth, Textured and Satin sheets BUT NOT on NylonPA or PP sheets.a. Nothing shown on LCD screen
M862.7 P14 W1
to simulate a PET/G print, which can be printed on Textured, Satin and NylonPA sheets BUT NOT on PP sheets. With a glue stick applied the usage of the Smooth sheet is possible and will warn the user.a. LCD should show for ~30 seconds
a. The LCD screen of step 5 shows and NEEDS to be confirmed on LCD
a. The LCD message is always shown, but it times out and will continue
Test 2
a. ideally enable some logging
Smooth
sheeta. LCD should show for ~30 seconds
b. To also check "old" gcode without
M862.7
, the check is also executed whenM115
is send during a print and shows following screen which DOESN'T timeout and NEEDS user interaction!i. To not delay host connections the
M115
doesn't execute the check when the printer is inactive.ii. Frankly at this moment new gcode will show the LCD message twice when
Always
is selected.