glue-viz / glue-ar

BSD 3-Clause "New" or "Revised" License
0 stars 1 forks source link

Add basic UI for scatter exporter #4

Closed Carifio24 closed 9 months ago

Carifio24 commented 9 months ago

This PR adds a basic UI for the scatter AR export tool. This is a pretty simple implementation. I would prefer to use the layer UUIDs rather than the labels, but it's simplest to store this using the same key as the list widget is displaying. We could do something like keep a map between UUIDs and indices, but that's an extra complication for really no gain - if you have multiple items with the same label, you won't know which one you're modifying the settings for anyway (and we obviously don't want to show the UUIDs). So for now, the philosophy here is that if you really shouldn't have multiple items with the same label.

This also tweaks the implementation of the scatter exporter so that we can set different theta and phi resolutions for each layer. This way, you could set a layer with a lot of small points to have a low resolution (if you aren't ever seeing them individually anyways), and/or layers with a few larger points to have a better resolution.