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

Adding the ability to turn StealthChop on and off (M569) #1609

Merged
merged 10 commits into from
Mar 7, 2021
Merged

Adding the ability to turn StealthChop on and off (M569) #1609

merged 10 commits into from
Mar 7, 2021

Conversation

radek8
Copy link
Contributor

@radek8 radek8 commented Feb 6, 2021

Adding the ability to turn StealthChop on and off (M569)

Adding the ability to turn StealthChop on and off (M569)
Addition of the index
@Guilouz
Copy link
Contributor

Guilouz commented Feb 7, 2021

What do you think about enable/disable machine settings on TFT screen ? To show or not show it, configurable in config.ini.

@kisslorand
Copy link
Contributor

What would be the benefit of it?

@radek8
Copy link
Contributor Author

radek8 commented Feb 7, 2021

I understand your questions.
TMC drivers have two operating modes, stealthChop and spreadCycle.
This PR allows you to switch the limits of these modes from the touch screen menu without having to switch to Marli mode.

@radek8
Copy link
Contributor Author

radek8 commented Feb 7, 2021

The Config.ini file is used to configure the TFT display parameters, not to configure Marlin.

@Guilouz
Copy link
Contributor

Guilouz commented Feb 7, 2021

The Config.ini file is used to configure the TFT display parameters, not to configure Marlin.

Yes I know, but some features disabled in Marlin like Linear Advance, Jerk (because use junction deviation), firmware retraction etc... still displays on TFT (with value 0 and can't be edited if are not enabled in Marlin). I think it's useful to choose which parameters you want hide / unmask on the TFT. Show which ones are actually editable.

@radek8
Copy link
Contributor Author

radek8 commented Feb 7, 2021

I'm not bothered by the options in the settings I don't have configured. I just don't click on them.
If someone clicks on the settings of a function that is not active, the printer returns a message - unknown command ...
Nothing serious will happen.
I'm afraid that disabling the standing settings options in Config.ini would be too much of an intervention in the code that no one will want to access.

@Guilouz
Copy link
Contributor

Guilouz commented Feb 7, 2021

I'm not bothered by the options in the settings I don't have configured. I just don't click on them.
If someone clicks on the settings of a function that is not active, the printer returns a message - unknown command ...
Nothing serious will happen.
I'm afraid that disabling the standing settings options in Config.ini would be too much of an intervention in the code that no one will want to access.

Me neither but I configure machines for people and they do not necessarily understand for certain parameters appear when the machine cannot manage it (like Linear Advance, Z Align etc). Especially since one of the last PR added all the parameters.

That's why I made this request, in the sense that only show the settings that are really modifiable.

@radek8
Copy link
Contributor Author

radek8 commented Feb 7, 2021

The PR you are talking about has added options from the ini file to the display parameters menu so that they can be edited from the display.
You are talking about options that set Marlin parameters and you want to optionally show and hide them.
I'm not a programmer and I don't want to express myself, but I can imagine that it won't be easy.
This would be a question about Bigtreetech, Guruathwal, Digant73, Kisslorand, Mehmetsutas ....
Kisslorand has already written his opinion :-)

@radek8
Copy link
Contributor Author

radek8 commented Feb 7, 2021

Are you making printers for your customers, and are you afraid that their own stupidity will change the parameters and they will want you to repair it?
If you have the correct default values for Marlin in Marlin and in the Config file. ini default values for TFT, then the remedy is very simple.

@Guilouz
Copy link
Contributor

Guilouz commented Feb 8, 2021

I mean to add possibility to hide some parameters in Machine settings screen. Parameters which are not used by the Machine used.

In exemple on a CR-10s Pro in Machine settings, I see the possibility to enable/disable Z Align but with this motherboard this function is not possible, same for Linear Advance, I see it, can enter a K-value. Which doesn't make sense to me because these functions are not enabled in Marlin.

Another exemple, in your PR you add StealthChop settings, I disable it in my Marlin firmwares but I would still see this parameter which will be useless on the machines I use.

It would be better to activate the parameters to display that one wishes or even better to activate these functions only when possible by Marlin.
Hope you understand what i mean.

@radek8
Copy link
Contributor Author

radek8 commented Feb 8, 2021

So open a new ticket with the Feature request

@radek8 radek8 closed this Feb 10, 2021
@rebel1
Copy link
Contributor

rebel1 commented Feb 11, 2021

Why do you closed this pr ?

@radek8
Copy link
Contributor Author

radek8 commented Feb 11, 2021

The parameter setting works great, but if I merge the PR and load it into the TFT, the display will freeze when I start printing and executing the G28 command.
I do not understand why. Printing is not affected by the adjustment. Execution of the G28 Command from the display will not freeze.

@radek8 radek8 reopened this Feb 21, 2021
@radek8 radek8 marked this pull request as draft February 21, 2021 17:20
@radek8 radek8 marked this pull request as ready for review March 6, 2021 19:47
@radek8
Copy link
Contributor Author

radek8 commented Mar 6, 2021

Fixed.
Now PR is fully functional and ready to merge.

@bigtreetech bigtreetech merged commit 4af62da into bigtreetech:master Mar 7, 2021
@radek8 radek8 deleted the M_569 branch April 17, 2021 20:54
jeffeb3 pushed a commit to V1EngineeringInc/BIGTREETECH-TouchScreenFirmware that referenced this pull request Nov 10, 2021
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.

5 participants