jukebox42 / Octoprint-PrusaMMU

An Octoprint plugin that adds MMU support.
GNU Affero General Public License v3.0
17 stars 2 forks source link

Support for Spoolman #46

Open SirFlip opened 6 months ago

SirFlip commented 6 months ago

Support for the Spoolman plugin https://github.com/mdziekon/octoprint-spoolman would be a great addition.

Then it would nicely work together with https://github.com/Donkie/Spoolman

jukebox42 commented 6 months ago

I'll take a look once I get this MK4 version stable. Hopefully they have similar hooks to the other spool managers I've hooked up.

GitSoks commented 2 months ago

+1 Support for Spoolman would be really awesome

crzykidd commented 2 months ago

+1 for Spoolman, Thanks for all your effort.

Llamrei80 commented 1 month ago

Any Update on this?

Great plugin, keep up the good work.

jukebox42 commented 2 weeks ago

Hey All,

I appreciate your patience. Finally got around to poking at this. I didn't realize I was signing myself up for spinning up another service lol. Took me a little bit longer to figure out how to capture the data (I really need to do some refactoring some day).

I'm not a regular user of spool man so I was hoping for a few testers to let me know if things are working as expected. Here is a patch version of the plugin with spool man support :).

A few notes:

To install:

Octoprint-PrusaMmu.zip

Thanks in advance!

DavyGD commented 2 weeks ago

Giving it a try!

graelo commented 1 week ago

Hi @jukebox42, thank you so much for this (and more generally the plugin).

I just successfully printed these 3 configs, and each time your plugin updated spoolman using the precise weights from the gcode:

Here are the screenshots for the 2 first configs

with tool 0 Screenshot tool 0
with tool 1 Screenshot tool 1

I have 2 remarks, but honestly not sure they are related the Spoolman integration

Your update works great for me, thanks!

HIH!

graelo commented 4 days ago

Hello, additional pieces of feedback.

Great:

The following are minor issues. In my view, there's no need to address them as part of this PR: they could be the topic of other issues. As you see fit.

Minor:

Less minor:

I hope it helps. Thanks again!