devhyper / open-video-editor

Open source Android video editor, built with Media3 and Jetpack Compose.
GNU General Public License v3.0
349 stars 15 forks source link

undo grey scale #32

Closed RustoMCSpit closed 6 months ago

RustoMCSpit commented 8 months ago

please design filters non-destructively until export

RustoMCSpit commented 7 months ago

closed as i made a new, broader issue

RustoMCSpit commented 7 months ago

ths doesnt work? @devhyper im confused, i thought you said we had non-destructive editing

devhyper commented 6 months ago

ths doesnt work? @devhyper im confused, i thought you said we had non-destructive editing

Click this: Screenshot 2024-03-14 192242

RustoMCSpit commented 6 months ago

ths doesnt work? @devhyper im confused, i thought you said we had non-destructive editing

Click this: Screenshot 2024-03-14 192242

ahhh, hitting greyscale itself doesnt undo it so i assume it's basing it off of a timeframe. regardless, theres no way to undo the greyscale mid-video. so if i greyscale at 5 seconds and then want to colorscale at 12 seconds i cant. does that make sense?

RustoMCSpit commented 6 months ago

please design filters non-destructively until export

ignore this, it is non-destructive

RustoMCSpit commented 6 months ago

like yk how you hit invert colors again and it goes back to normal, but grey doesnt. either have everything work as i described above or have it do as you do with the invert colors, having two standards is bad

devhyper commented 6 months ago

Yeah sorry for the confusion. When you click the filter twice it will just apply the filter again, so invert colors just inverts the already inverted colors back. I'll leave it as is now, because when composition previewing lands in Media3, I will implement the ability to specify filter lengths. This will also allow for the grayscale certain part and invert certain part you mentioned.

RustoMCSpit commented 6 months ago

Yeah sorry for the confusion. When you click the filter twice it will just apply the filter again, so invert colors just inverts the already inverted colors back. I'll leave it as is now, because when composition previewing lands in Media3, I will implement the ability to specify filter lengths. This will also allow for the grayscale certain part and invert certain part you mentioned.

when will that be implemwnted and what is it simply?

devhyper commented 6 months ago

Basically the app uses the Android Media3 library as a backend to process and play the video. Currently, the library has this thing called Composition, which allows adding multiple videos together, cutting them into segments, adding effects to certain segments, etc. However the player in the library cannot play this composition yet, they are working on it in their main branch though. My guess is that it will come in their library version 1.4.

55

RustoMCSpit commented 6 months ago

Basically the app uses the Android Media3 library as a backend to process and play the video. Currently, the library has this thing called Composition, which allows adding multiple videos together, cutting them into segments, adding effects to certain segments, etc. However the player in the library cannot play this composition yet, they are working on it in their main branch though. My guess is that it will come in their library version 1.4.

55

how can yoy put stuff in the next milestone if thatll take months to come out

devhyper commented 6 months ago

The stuff in the milestones don't require composition. Everything that does is tracked by #55