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

Falashforge 5M network support #4546

Merged
merged 6 commits into from
Mar 22, 2024
Merged

Falashforge 5M network support #4546

merged 6 commits into from
Mar 22, 2024

Conversation

iherbak
Copy link
Contributor

@iherbak iherbak commented Mar 18, 2024

The build_orca.yml should not be merged as it is tailored to my repo, but the rest of the commits

Thanks

@SoftFever
Copy link
Owner

Thank you @iherbak
I haven't done the code review yet.
Could you revert the changes in the workflow and README files on your side?
GitHub doesn't support the selective merge option, unfortunately.

@SoftFever
Copy link
Owner

@Noisyfox I noticed you are working on SimplyPrint connection.
Can you help review this PR too?

@iherbak
Copy link
Contributor Author

iherbak commented Mar 18, 2024 via email

@Noisyfox
Copy link
Collaborator

LGTM

@Lexapskov
Copy link

We need that stuff

@SoftFever
Copy link
Owner

I have started the build.

Looking for testers who have this machine to test.

@SoftFever SoftFever added the Community testers wanted Looking for community testers and feedback label Mar 19, 2024
@jaimbo
Copy link

jaimbo commented Mar 19, 2024

I have started the build.

Looking for testers who have this machine to test.

I can give it a go

@jaimbo
Copy link

jaimbo commented Mar 19, 2024

Adding printer works correctly, uploading sending gcode over the network also working correctly.

When specifying a directory for the uploaded gcode, this seems to be ignored by the printer and the file is put into the root of the onboard storage. The gcode still appears to be working but when a direcory is added, the printer won't show a working preview of the print on the screen.

Connection Success

EDIT: Correction, when you specify slashes in the gcode to upload, if you choose to only upload (not print) the file will not appear in the printers onboard storage.
If you choose upload and print, the printer will act like it is starting a print, it will say it's preheating but the nozzle/bed doesn't appear to be heating and if you go to the print info, there are no details there. As above, the file will also be missing from the onboard storage.Seems directories aren't currently working in any fashion.

image
image

@glizzyglizz69
Copy link

I have started the build.

Looking for testers who have this machine to test.

I'm available to test - Just installed

@SoftFever
Copy link
Owner

Adding printer works correctly, uploading sending gcode over the network also working correctly.

When specifying a directory for the uploaded gcode, this seems to be ignored by the printer and the file is put into the root of the onboard storage. The gcode still appears to be working but when a direcory is added, the printer won't show a working preview of the print on the screen.

Connection Success

EDIT: Correction, when you specify slashes in the gcode to upload, if you choose to only upload (not print) the file will not appear in the printers onboard storage. If you choose upload and print, the printer will act like it is starting a print, it will say it's preheating but the nozzle/bed doesn't appear to be heating and if you go to the print info, there are no details there. As above, the file will also be missing from the onboard storage.Seems directories aren't currently working in any fashion.

image image

Does it work if you don't change the upload folder?
Many klipper system don't support uploading g-code to a subfolder.

I don't have this machine.
So I leave @iherbak @FlashforgeOfficial to answer.

@jaimbo
Copy link

jaimbo commented Mar 21, 2024

Does it work if you don't change the upload folder? Many klipper system don't support uploading g-code to a subfolder.

I don't have this machine. So I leave @iherbak @FlashforgeOfficial to answer.

Yes, it works correctly if you don't specify a subfolder.

There doesn't seem to be any manual way to create a folder on the internal storage either, though the UI on the screen for starting a print will show/print from directories on a connected usb storage device.

@iherbak @FlashforgeOfficial If it's not possible to create a folder on the onboard storage, can we change filename window that appears to not suggest it? Would it also be possible to sanitize any slashes in the filenames so that the print will upload and start successfully if someone does try it?

Also, this might not be the correct place for this, but it would be handy if you could choose to level the bed before starting the print on OrcaSlicer, just like you can when starting print from the screen.

@iherbak
Copy link
Contributor Author

iherbak commented Mar 21, 2024 via email

Copy link
Owner

@SoftFever SoftFever left a comment

Choose a reason for hiding this comment

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

Looks good
Thank you

@SoftFever
Copy link
Owner

Does it work if you don't change the upload folder? Many klipper system don't support uploading g-code to a subfolder.
I don't have this machine. So I leave @iherbak @FlashforgeOfficial to answer.

Yes, it works correctly if you don't specify a subfolder.

Thanks

@SoftFever SoftFever merged commit f1c91dc into SoftFever:main Mar 22, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community testers wanted Looking for community testers and feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants