Jump to content
markwal

Plugin dev: where did the name go?

Recommended Posts

Posted · Plugin dev: where did the name go?

In current master, it looks like save local no longer proposes the name of the first mesh as the filename. Back to typing in a name. However, my plugin was also trying to use the following to propose an initial file name, but it no longer works. Did the property move? If so, where to?

 

for n in BreadthFirstIterator(node):   if n.getMeshData():       fileName = n.getName()       if fileName:           break

 

where node is passed into OutputDevice.requestWrite

Share this post


Link to post
Share on other sites
Posted · Plugin dev: where did the name go?

Before the settings rework branch merge, fileName would be the name of a mesh (the file name of the stl file). It appeared to be the first stl that was opened.

Share this post


Link to post
Share on other sites
Posted · Plugin dev: where did the name go?

Ah right. It could be that we broke a bunch of stuff while doing this. The code does use the first name it finds as the name to write to the output device, but if no node has a name set, it returns None.

This proves that it's always a good idea to not completely trust the data that you receive from other parts of the application and build in a check if the received name is not set.

Share this post


Link to post
Share on other sites
Posted · Plugin dev: where did the name go?

I agree. But now what is the right way to choose a good default name? It's rather annoying to force the user to type the name they just opened if they'd like the gcode name to match the stl.

Share this post


Link to post
Share on other sites
Posted · Plugin dev: where did the name go?

That's the thing with master; This is where we do new feature development. This means you get cutting edge features, but with no guarantee that it actually works.

I will have a look if I can make a quick fix for this though.

Share this post


Link to post
Share on other sites
Posted · Plugin dev: where did the name go?

Oh I see now, it's just a bug. I thought you'd moved the property and I just couldn't figure out where it went. Thanks for the fix.

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Our picks

    • Ultimaker Cura 4.0 | Stable available!
      Ultimaker Cura 4.0 is mainly focused on the improved user interface and cloud integration.
      As always, we want to collect your user feedback for this release. If there are any improvements you can think of, feel free to mention it here and help us to shape the next release.
      • 96 replies
×
×
  • Create New...

Important Information

Welcome to the Ultimaker Community of 3D printing experts. Visit the following links to read more about our Terms of Use or our Privacy Policy. Thank you!