Updated LAN Only model download for filament tracking #80
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New to GIT so hope this is right.
I was having issues where the print history or filament tracking was not working on my P2S in LAN Only mode, Read through the logs and stepped through the code to find issues with the model download was trying to grab paths it didn't have access to from the printers internal storage.
Changes:
Corrected LAN Only gcode file download to fix filament tracking.
Added path filtering to filename only to fix cases where there are unexpected paths.
Added retry attempts to prevent race condition where files were not fully saved on the printer before trying to grab them.
If a file still can't be downloaded, list out the files it could find in the cache directory for debugging.
Required printer option: Store sent files on external storage
Tested on P2S with AMS 2 Pro
Maybe related to issues:
#41
#78