-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add initial support for the AnkerMake M5C #145
Conversation
I just realized that sending g-code from the command line results in an error - will investigate further. |
This change allows to monitor and send MQTT messages from/to the AnkerMake M5C on the command line as well as observe the printer status via the Web GUI. It is also possible to send print jobs from your slicer to the printer without using AnkerMake Slicer or Studio (see README.md). Due to the M5's additional hardware (camera, light) still being expected to exist, the Web GUI will continuously wait for a camera connection. This should be fixed in the future.
0d0c88b
to
d9af577
Compare
Found the issue! Apparently the BTW, I also adjusted README.md a bit to mention the M5C. |
One more observation - the PPPP badge in the web interface will currently not turn green for the M5C because, after reading the code, this is actually the status of the video connection. So the PPPP connection may be established correctly after all and sending gcode files will work just fine. |
Maybe we should add a check for the badge that does a health check on PPPP for M5C to turn the status green in absence of the video feed |
I may take a look into that as time permits. However, as it supposedly does not affect the overall functionality I assume this can be done via a separate PR. |
Any review comments for this PR? Are there things to change before it can be merged? |
As there was no feedback yet to the previous commits anyway, I ended up adding the feature suggested by @billyjbryant. I.e. the PPPP status is now correctly displayed for both M5 and M5C by expecting a PPPP message at least every 3 seconds. This makes the badge turn green, otherwise it is displayed as orange. The 3 seconds are a valid assumption IMO because the printer sends at least an AlivePkt every second. This latest commit also adds conditions around parts of the web interface which are not applicable to the M5C. So M5C users will not see the rotating lines in the video area anymore, nor the controls for the camera or the VIDEO badge. I have tested this with my M5C (which "luckily" drops the Wifi connection very often when connected to my Unifi AP), but tests with an M5 would be appreciated to see if things work the same as before (except for the added VIDEO badge). |
Now the PPPP badge just shows the status of the PPPP connection, and this also works for the AnkerMake M5C. The new VIDEO badge now has the same behavior as the original PPPP badge in that it shows the status of the video stream.
222098f
to
1089b49
Compare
I opened this PR almost one month ago, and while I admittedly added more changes later on, there was no comment to the specific code changes or prospects for when this can be merged from any of the maintainers of this great piece of software. In the meantime I direct people with various issues to my own integration branch, but this cannot be a permanent situation IMO. Any thoughts about the changes and when this can be merged? Or is this project not actively maintained and developed further anymore? |
Deployed this branch this morning. Works like a charm, way better than manually exporting gcode and opening Ankermake Studio! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code gets the Ankermake M5C perfectly working, got it running on my own setup
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very good very work
still haven't tested sending prints over from ps so no approval yet
but very good job
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uploading files from prusaslicer gets stuck on "RESOLVING ADDRESS".
Uploading files: no worky :(
Everything else: yes worky :)
This is very likely an issue of PrusaSlicer and not of this PR or ancertctl in general, see prusa3d/PrusaSlicer#10944 . I saw the same issue a while ago but the latest versions of PS seem to have fixed it. |
I'm running PS 2.8, so it isn't that. tldr: fck you ankermake |
Was this work moved somewhere? |
This and other PRs are now incorporated in https://github.com/anselor/ankermake-m5-protocol/tree/exiles-1.1 . Unfortunately, I have to admit that the documentation is still not up to date and matching the now recommended way (which is a simple login and printer search via the web interface). |
This PR fixes #137 and allows to monitor MQTT messages of the AnkerMake M5C on the command line as well as observe the printer status via the Web GUI. It is also possible to send print jobs from your slicer to the printer without using AnkerMake Slicer or Studio (see README.md). The M5's additional hardware is still expected to exist, therefore the Web GUI will still pretend to wait for a camera connection, for example. This will have to be fixed in the future.
Note that I was only able to test with my M5C but not with an M5 for unwanted side-effects.
Eventually, if this patch is confirmed by others to work fine, we should add some related information to the README file.