search

vanvalenlab / kiosk-frontend

DeepCell web application built using NodeJS, Express, React, and Webpack.
Other
1 stars 0 forks source link

Replace `ScaleForm` with new `ResolutionDropdown`. #171

Closed willgraf closed 3 years ago

willgraf commented 3 years ago

As discussed in #165, the ScaleForm is a bit too unintuitive for users. It's not clear what the correct "scale" should be. This PR replaces the ScaleForm with a manual dropdown with common microns-per-pixel values (and the zoom level used with them). This way, the user can select if their data is 40x or 10x instead of trying to guess at the correct scale.

I think there is still room for a manual input form for image mpp, but for now, I think that the ballpark resize values will be helpful and intuitive.

Fixes #165

willgraf commented 3 years ago

I've deployed this on deepcell.org and tested with a few example files and resolution inputs.

Viewing the backend logs, I can see that the consumer is reading the scale properly and resizing the image as expected.

willgraf commented 3 years ago

That makes sense to me. Now the content lines up a bit better on the bottom too!

I've redeployed it on the site. LMK if you see anything.