AnSyn / ansyn

End-to-end solution for overhead image analysis
https://ansyn.io/
MIT License
57 stars 8 forks source link

image processing #271

Open carmelorr opened 7 years ago

carmelorr commented 7 years ago

image processing is divided to 3 levels:

histogram and fully automatic functions basic- automatic contrast and brightness basic manual functions- brightness, contrast, gamma and saturation

eldadcohen1 commented 7 years ago

Talk with the board on this issue Required: @idanbarak @ronenru @OmKoren

idanbarak commented 7 years ago

@carmelorr the tools menu will be smaller (narrower) than the other menus on the sidebar and will include many other tools other than "image processing" (it is all ready include the "hover mouse" button). we cant use all of the menu to the manual image processing tools.

OmKoren commented 7 years ago

@eldadcohen1 Regarding the automatic image processing functions, I suggest that we start by implementing the simplest one: histogram equalization. Once we see it is doable - we can implement more sophisticated algorithms. Make sure to re-calculate it every time the pixels change (frustum, pan/zoom,...)

Here is a good explanation / implementation of histogram equalization in JS: http://phg1024.github.io/image/processing/2014/02/26/ImageProcJS4.html

OmKoren commented 7 years ago

@carmelorr I think that your design lacks the automatic image processing buttons.

carmelorr commented 7 years ago

Automatic processing-

Automatic processing will include- histogram, sharpness, (and gamma, brightness?)

One click on the icon (or toggle) will make a momentary processing (only observed area) Two clicks on the icon (or toggle) will make a constant processing.

Manual processing-

Manual processing will turn off the automatic processing (always?)

It will include- sharpness, histogram, gamma, brightness, saturation and contrast.

carmelorr commented 7 years ago

image

idanbarak commented 7 years ago

@carmelorr i think we dont need the expand/collapse buttons in the manual processing menu. we can expose all of the sliders permanently

carmelorr commented 7 years ago

image

idanbarak commented 7 years ago

@carmelorr add reset

carmelorr commented 7 years ago

image

idanbarak commented 7 years ago

@carmel i labeled it "ready for development" but please change gray divider to a bright option

carmelorr commented 7 years ago

image @idanbarak

carmelorr commented 7 years ago

Please add tool tip- "Manual Image Enhancement"

idanbarak commented 7 years ago

@carmelorr i think we should consider some kind of visual representation (or preview) for the histogram slider. please see "picsplay" android app for inspiration

idanbarak commented 7 years ago

picsplay-fx-photo-editor-13-300x535

carmelorr commented 7 years ago
5 7
idanbarak commented 7 years ago

@carmelorr nice

carmelorr commented 7 years ago

As decided- at the moment the manual functions will cancel the auto ones and vise - versa. Also- please note that the manual enhancement functions will apply to each overlay (as opposed to imagery), and will not be saved. For later on- maybe save them per case?