Unidata / IDV

The Integrated Data Viewer (IDV) from Unidata is a framework for analyzing and displaying geoscience data.
http://www.unidata.ucar.edu/software/idv/
Other
80 stars 37 forks source link

Magnification sliders not properly linked #124

Open dopplershift opened 7 years ago

dopplershift commented 7 years ago

When adjusting the image magnification, even though the image size and magnification are linked, if I adjust magnification using the right (y?) slider, magnification/image size along only one of the dimensions is adjusted. If I use the left (x?) slider, both are adjusted. See animation:

idv-bug

donmurray commented 7 years ago

That is by design. Element mag acts differently than line magnification because some images are oversampled in the x direction.

buddhaandthebees.net

On Thu, Apr 20, 2017 at 2:43 PM, Ryan May notifications@github.com wrote:

When adjusting the image magnification, even though the image size and magnification are linked, if I adjust magnification using the right (y?) slider, magnification/image size along only one of the dimensions is adjusted. If I use the left (x?) slider, both are adjusted. See animation:

[image: idv-bug] https://cloud.githubusercontent.com/assets/221526/25251777/b2a1c5be-25d7-11e7-865a-4d4f0c57e172.gif

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Unidata/IDV/issues/124, or mute the thread https://github.com/notifications/unsubscribe-auth/AAmEpIjvrBuuyvhhEom9XjBwBwEgrVMvks5rx8OEgaJpZM4NDjWe .

dopplershift commented 7 years ago

I would suggest then, that some labels for those sliders would go a long way towards making that clearer--I had no idea whatsoever that's what the different sliders were for. Also, based on that explanation, I'm unclear why changing the "line magnification" would change the "element magnification", but then not the other way.

I also found the following pretty confusing:

idv-magnify

donmurray commented 7 years ago

Or, maybe it's time for a pardigm shift. This widget was intended mainly for McIDAS-X users who know about such things. I would suspect that most other users don't know how to use it anyway. (for example, the NxM is Y by X (linesXelements), not the normal X by Y that most geometries use). I'm not saying it's right or wrong, I'm just giving you context of why it does what it does. Feel free to discuss this with SSEC since it's mainly there for Mc-V users - I have no skin in the game.

buddhaandthebees.net

On Thu, Apr 20, 2017 at 4:10 PM, Ryan May notifications@github.com wrote:

I would suggest then, that some labels for those sliders would go a long way towards making that clearer--I had no idea whatsoever that's what the different sliders were for. Also, based on that explanation, I'm unclear why changing the "line magnification" would change the "element magnification", but then not the other way.

I also found the following pretty confusing:

[image: idv-magnify] https://cloud.githubusercontent.com/assets/221526/25254624/bb804c58-25e3-11e7-82fa-b1ebea6983f7.gif

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Unidata/IDV/issues/124#issuecomment-295937851, or mute the thread https://github.com/notifications/unsubscribe-auth/AAmEpA7RSfJRZXnsA9FfRPQA2J8oLZyvks5rx9e8gaJpZM4NDjWe .

jeffmcwhirter commented 7 years ago

I think I developed that widget and I never did understand it :)

Jeff

On Thu, Apr 20, 2017 at 9:01 PM Don Murray notifications@github.com wrote:

Or, maybe it's time for a pardigm shift. This widget was intended mainly for McIDAS-X users who know about such things. I would suspect that most other users don't know how to use it anyway. (for example, the NxM is Y by X (linesXelements), not the normal X by Y that most geometries use). I'm not saying it's right or wrong, I'm just giving you context of why it does what it does. Feel free to discuss this with SSEC since it's mainly there for Mc-V users - I have no skin in the game.

buddhaandthebees.net

On Thu, Apr 20, 2017 at 4:10 PM, Ryan May notifications@github.com wrote:

I would suggest then, that some labels for those sliders would go a long way towards making that clearer--I had no idea whatsoever that's what the different sliders were for. Also, based on that explanation, I'm unclear why changing the "line magnification" would change the "element magnification", but then not the other way.

I also found the following pretty confusing:

[image: idv-magnify] < https://cloud.githubusercontent.com/assets/221526/25254624/bb804c58-25e3-11e7-82fa-b1ebea6983f7.gif

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Unidata/IDV/issues/124#issuecomment-295937851, or mute the thread < https://github.com/notifications/unsubscribe-auth/AAmEpA7RSfJRZXnsA9FfRPQA2J8oLZyvks5rx9e8gaJpZM4NDjWe

.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Unidata/IDV/issues/124#issuecomment-296031695, or mute the thread https://github.com/notifications/unsubscribe-auth/ABonT0o78sAwKD481xQ8xotb8TEV5l6Eks5ryBvtgaJpZM4NDjWe .