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

Update Turkish Language / Board SD Printing: Show Actual Z Coordinate as Layer - Watch pause and cancel action - Bypass blocking operations on print cancel / Fix M27 time parameter /M701-M702 load/unload filament support #1272

Merged
merged 31 commits into from
Nov 27, 2020

Conversation

mehmetsutas
Copy link
Contributor

@mehmetsutas mehmetsutas commented Nov 19, 2020

  • Update outdated Turkish translation

  • During SD printing status screen showed 0mm as layer throughout the print. Actual Z position is shown which is more informative than 0mm

  • Pause, paused and cancel keywords are added to the hostactions to watch the boardsd prints. Now when prints are canceled by host pc, it can be detected and LCD reverts to the statusmenu from printingmenu. Cancel action on printabort will also be added to Marlin.

  • M524 processing was deferred by blocking operations such as extruder and bed heating processes. Break gcodes added before M524 in order to break the blocking operations and speed up the print cancel when printing from boardsd.

  • Miliseconds were passed as parameter to M27 gcode which expects seconds as parameter. Fixed.

  • Add support for M701-M702 Marlin load/unload filament gcodes. A new configuration option added for M701/702 support. A subpage added to the preaheat menu. Preheat menu moved to the main page in place of extrude menu. Extrude menu moved under move menu. After selecting the preheat option printer starts to heat on the next newly added page either filament load or unload option can be chosen or can exit without filament load/unload. Accordingly M701 or M702 gcode is sent to Marlin and Marlin handles the filament load/unload according to its setup on configuration_adv.h

resolves #1135
resolves #834
resolves #367
resolves #368

@oldman4U
Copy link
Contributor

You are a gift from heaven!!!

Thank you👍🏻

Copy link
Contributor

@traffic-light traffic-light left a comment

Choose a reason for hiding this comment

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

I scrolled trough your code. See the individual comments.

In short:

  • Use curly braces, even when it's one line
  • Place curly braces on its own line. Example:
}
Else
{
  • Create a space after if, while, for etc. : if (what == true)
  • Line up you're code with the rest of it

Thank you

@traffic-light
Copy link
Contributor

traffic-light commented Nov 19, 2020

It seems something went wrong with my suggestions. I hope it's clear enough.

Oh well. Thank you for your work!

@Maker-M
Copy link

Maker-M commented Nov 20, 2020

Excellent news ! Forgive my ignorance, but what should we do? Is there a file to copy directly or a patch? Do I have to replace each red line with the corresponding green line by recompiling the affected files?

@oldman4U
Copy link
Contributor

Hi Maker-M.

Are you mehmetsutas?

@mehmetsutas
Copy link
Contributor Author

@traffic-light I will check your suggestions.

@Maker-M You may copy paste the changes for now. But you should better wait for the changes to be merged into the source. Meanwhile people may have different comments and point out some problems with the change if there are. After merge you can download the firmware again, compile and use it.

@oldman4U he is not. :)

@Maker-M
Copy link

Maker-M commented Nov 20, 2020

@ feu de circulation Je vérifierai vos suggestions.

@ Maker-M Vous pouvez copier-coller les modifications pour le moment. Mais vous devriez mieux attendre que les modifications soient fusionnées dans la source. Pendant ce temps, les gens peuvent avoir des commentaires différents et signaler certains problèmes avec le changement, le cas échéant. Après la fusion, vous pouvez télécharger à nouveau le micrologiciel, le compiler et l'utiliser.

@ oldman4U il ne l'est pas. :)

Great ! Thanks for your answers.
Sorry oldman4U, i'm not...

@oldman4U
Copy link
Contributor

oldman4U commented Nov 20, 2020

Ahhhh, it is compiling;-)))

We had cases where users have been logging in using different devices which resulted in different names - so this is why I asked.

@mehmetsutas Thank you very much for contributing. Feel free to solve more long lasting issues #1218 or adding great features #1170 ;-)!!!

@blueeagle69
Copy link

You are a genius.
Layer info finally working when printing from onboard media.

I have noticed though that if no ABL is active (no mesh stored) the ABL icon doesn't show.
If I go to Marlin mode and run ABL then return to TFT the ABL icon appears.

I don't think this is related, as I'm sure this was mentioned elsewhere.
But thought I'd mention it.
Great work on getting the layer info working!

@mehmetsutas mehmetsutas changed the title Update Turkish Language / Board SD Printing: Show Actual Z Coordinate as Layer - Watch pause and cancel action - Bypass blocking operations on print cancel / Fix M27 time parameter Update Turkish Language / Board SD Printing: Show Actual Z Coordinate as Layer - Watch pause and cancel action - Bypass blocking operations on print cancel / Fix M27 time parameter /M701-M702 load/unload filament support Nov 20, 2020
@mehmetsutas
Copy link
Contributor Author

@oldman4U it is easier to solve when the coder personally encounter the problem. But I will have look at the list.

I have added M701/702 load/unload gcode support. Will work on M600 and power loss recovery from boardsd.

Mehmet Sutas and others added 2 commits November 20, 2020 16:16
@oldman4U
Copy link
Contributor

Hi.

The problem is, that most of the developers in the past added a feature and then they disappeared. Most likely because of a completely chaotic handling of this repository. Some users including me, try to get the things organised again, by adding precompiles firmware which works and which are not several months old, by updating the ReadMe and removing things which are not working anymore. For me the most critical issue is the lack of engagement from BTT, so we try to work around this as much as we can. Thats one of the reasons we made the FR and Bug list, otherwise we would have hundreds of tickets with a search function which only works in parts and important issues and features on page 2 and later - and who reads more than the first page. So we got reports covering the same issues and the same time developers silently working on the same things.

Long story short. It would be great if we could convince you to stay a bit longer, in case you are interested and in case you have a bit time to spare.

Thank you

@mehmetsutas
Copy link
Contributor Author

Created a pull request on Marlin for reporting the cancelled SD print.

MarlinFirmware/Marlin#20223

@mehmetsutas
Copy link
Contributor Author

Hi.

The problem is, that most of the developers in the past added a feature and then they disappeared. Most likely because of a completely chaotic handling of this repository. Some users including me, try to get the things organised again, by adding precompiles firmware which works and which are not several months old, by updating the ReadMe and removing things which are not working anymore. For me the most critical issue is the lack of engagement from BTT, so we try to work around this as much as we can. Thats one of the reasons we made the FR and Bug list, otherwise we would have hundreds of tickets with a search function which only works in parts and important issues and features on page 2 and later - and who reads more than the first page. So we got reports covering the same issues and the same time developers silently working on the same things.

Long story short. It would be great if we could convince you to stay a bit longer, in case you are interested and in case you have a bit time to spare.

Thank you

I understand and I will try to support. It is hard for me to solve the problems of the features that I don't use but I will surely share any fix or enhancement that I make.

@oldman4U
Copy link
Contributor

oldman4U commented Nov 21, 2020

Hi.

Could you please add the following to the description of this PR. Thank you

resolves #1135
resolves #834
resolves #367
resolves #368

@ETE-Design
Copy link

@mehmetsutas Reallyd great to se you here, and nice finally filament chance is added... Mabye your the guy who can look at #1264 and make it?

@ETE-Design
Copy link

@bigtreetech Seem's like it's time to merge a Lot again...

@oldman4U
Copy link
Contributor

@bigtreetech Seem's like it's time to merge a Lot again...

Let's pray and hope for the best. It is almost a bit like BTT does not want this to be a successful repository.

@bigtreetech bigtreetech merged commit e883afa into bigtreetech:master Nov 27, 2020
jeffeb3 pushed a commit to V1EngineeringInc/BIGTREETECH-TouchScreenFirmware that referenced this pull request Nov 10, 2021
… as Layer - Watch pause and cancel action - Bypass blocking operations on print cancel / Fix M27 time parameter /M701-M702 load/unload filament support (bigtreetech#1272)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
7 participants