-
-
Notifications
You must be signed in to change notification settings - Fork 465
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
Feature request. Adjustable delay for file transfer. #1051
Comments
What file transfert are you refering to? And what version of ESP3D are you using? |
Old style M28/M29 file transfer protocol, ESP3D2.1.x |
This protocol is no more supported https://esp3d.io/ESP3D/Version_2.1.X/documentation/sdtransfert/ |
Like big corpo discontinuing functional products... |
I am not Big corpo, I am not paid for what I do |
Won't fix |
sorry I disagree my comment was not rude I answered to your feature request and my comment had a link that explained why it is not supported, it was not rude but explained. |
Absolutely not bad day, you just misinterpreted my comment which was meant to have humoristic attitude. Especially when my work does exist mainly because big fishes in industrial business say they no longer support something and offer expensive upgrade so we are here to fix things. |
Discussed in #1010
Originally posted by A13xund3r March 19, 2024
I was repairing BTT GTR V1 mainboard at work and took opportunity to check how does ESP3D communicate during file upload. So this is scope screenshot, yellow line is data sent by ESP3D, blue line is GTR response.
As you can see GTR respond literally immediately after ESP finish line, I didn't sniffed what it is but I'm 100% sure it is "OK". So why ESP wait 20ms before it send another line? I suspect it is safety measure to give mainboard time to process data it just received but I expect it is already processed when it reply "OK". Also BTT GTR is a monster comparing to any Atmel based controller so I think we can play with shorter or no delay at all. At least it could be available as adjustable option in settings.
The text was updated successfully, but these errors were encountered: