allen-cell-animated / agave

Other
34 stars 6 forks source link

add option to keep current agave settings when loading #160

Closed toloudis closed 8 months ago

toloudis commented 8 months ago

If loading a new volume, offer a LoadDialog checkbox to keep or discard current volume appearance settings. Default is to keep settings. This is to make it efficient when loading consecutive images of same channels and approximate intensity ranges. Unchecking the box will present the previous behavior, where every setting goes back to a default.

Example: load a 9-channel variance dataset volume, enable a set of channels. Customize channel colors. Adjust thresholding. Then load another 9-channel variance dataset volume. Same channels should be enabled, with same colors etc.

ShrimpCryptid commented 8 months ago

It seems like any changes to the ROI on the LoadDialog are overridden when keeping the existing settings. Is this intended behavior?

UI has an existing ROI, and the LoadDialog has a different ROI set: image

Once the image is loaded, it keeps the old ROI range, ignoring what was set in LoadDialog: image

toloudis commented 8 months ago

It seems like any changes to the ROI on the LoadDialog are overridden when keeping the existing settings. Is this intended behavior?

UI has an existing ROI, and the LoadDialog has a different ROI set... Once the image is loaded, it keeps the old ROI range, ignoring what was set in LoadDialog...

I think this is actually the correct/intended behavior. The ROI in the loading dialog is letting you select a subset of what's in the original file to load. Once that is loaded, that volume becomes the ENTIRE volume that the app knows about. So if your loading ROI starts at x0, y0, z0 and spans a size of dx, dy, dz, then the LOADED volume (and the ROI you see in the appearance panel ROI section) will start at 0,0,0 and be of size dx,dy,dz.

The saved settings of ROI in the Appearance panel preserve the relative ROI selection relative to the volume that was loaded.