search

balintlaczko / VideoAnalysis

Non-realtime video analysis, exporting motiongrams and various quantitative features of movement in the video file.
https://www.uio.no/ritmo/english/research/labs/fourms/downloads/software/VideoAnalysis/
1 stars 0 forks source link

Add (fix) option to remember cropping settings per item when working with a folder of videos #39

Closed balintlaczko closed 4 years ago

balintlaczko commented 4 years ago

It is already there in the [crop-stuff] below the [umenu] but as of now it always stores the last drawn rectangle with the currently recalled item index - that should be fixed. This should also exist as an option, alongside with the other, which is applying the same cropping settings for each file in a folder.

aleksati commented 4 years ago

In what way do you suggest the [crop-stuff] dim storage system should be fixed or altered? As you say, it does work. The only problem I´ve identified is some scheduling issues (in the cropping section) when exporting which messes with the video dims.

balintlaczko commented 4 years ago

Yes, the saving part seems to work now, I remember it didn't look right a month ago, but maybe it was OK already. Now the problem is just the the option itself is not implemented, so cropping switches off with each new video, and also items in the folder which don't yet have a cropping rectangle are not refreshing the cropping display, but those will be relatively easy to implement - hopefully. :)

aleksati commented 4 years ago

True! I can try to look more into this during the week.

aleksati commented 4 years ago

This is now implemented and working pretty nicely. Refreshes back to "spoof" dim if cropping is not specified on a video, and chooses the LAST drawn cropping rect as dim for all videos if the other option is selected. I disconnected the trigger which turns off the "cropping" box when changing videos in a folder. Do we ever want this box to turn off between videos? We can discuss this later I guess.

However, there might be an issue with videos that don´t have 320x240 dim when exporting. I need you to take a look at it on thursday when we merge, @balintlaczko .

aleksati commented 4 years ago

This works, need some bug-fixes when exporting. Will make seperate issue for this.