10.04.2024, 17:26
Pages: 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58
10.04.2024, 17:59
10.04.2024, 18:12
Did a quick test and it does fix the problem. ![Smile Smile](https://forum.selur.net/images/smilies/smile.png)
Cu Selur
![Smile Smile](https://forum.selur.net/images/smilies/smile.png)
Cu Selur
10.04.2024, 18:13
(10.04.2024, 15:10)Selur Wrote: [ -> ]Quote:Could you fix this issue ?There is nothing to fix. That is a text field, Hybrid does no validation for it.
This is not exact.
For example if I check "Chroma Smoothing" and set it to: [0.2, 0.7, 0.9, 0.5]
If I uncheck "Chroma Smoothing" the list change in: [0.2, 0.7, 0.9, 0.1]
If I recheck the list is still [0.2, 0.7, 0.9, 0.1] so the change of last parameter to "0.5" was lost.
Only "Darkness" is not showing this behaviour, but "Chroma Smoothing", "Chroma Stabilizer" and "Tweaks" are affected by this problem.
Please check.
Dan
10.04.2024, 18:22
Ah, I forgot to reset the Darkness text field, when Darkness is disabled.
-> adjusted, this is how I intended it to be.
Not a bug, it's a feature. ![Smile Smile](https://forum.selur.net/images/smilies/smile.png)
(updated download)
Cu Selur
Ps.: Especially for filters where the developer frequently changes the default values this seems like a good behaviour to keep.
-> adjusted, this is how I intended it to be.
![Big Grin Big Grin](https://forum.selur.net/images/smilies/biggrin.png)
![Smile Smile](https://forum.selur.net/images/smilies/smile.png)
(updated download)
Cu Selur
Ps.: Especially for filters where the developer frequently changes the default values this seems like a good behaviour to keep.
10.04.2024, 18:50
I don't agree.
I save my default values using Config->save all
In this way the old defaults reset the saved defaults.
Please remove this feature at least for this filter.
It is not necessary the unsaved changes will be lost when the Hybrid is closed.
Thanks,
Dan
I save my default values using Config->save all
In this way the old defaults reset the saved defaults.
Please remove this feature at least for this filter.
It is not necessary the unsaved changes will be lost when the Hybrid is closed.
Thanks,
Dan
10.04.2024, 18:56
I think it's a bad idea, but I changed the behaviour.
(updated the download)
Cu Selur
(updated the download)
Cu Selur
10.04.2024, 19:28
Thanks for the update! ![Smile Smile](https://forum.selur.net/images/smilies/smile.png)
Dan
![Smile Smile](https://forum.selur.net/images/smilies/smile.png)
Dan
11.04.2024, 18:15
(10.04.2024, 08:20)Dan64 Wrote: [ -> ]It could be useful if the Color Panel in vsViewer will be extended to display the values of HSV, currently only RGB and YUV values are shown.
To get the correct range to adjust, at the moment, I save a screenshot of the image with the wrong colors and open it in Paint.NET and use the color picker tool to see the HSV values.
Hello Selur,
what do you think to add in vsViewer Color Panel the HSV values ?
the algo to convert the RGB in HSV is quite simple, see: https://www.geeksforgeeks.org/program-ch...lor-model/
Thanks,
Dan
11.04.2024, 18:47
Doesn't really seems useful, since Vapoursynth itself doesn't use HSV.
Also, since the video usually isn't RGB, so not sure whether using the Display values is accurate:
When using the DisplayRGB values as base
https://github.com/Selur/vsViewer/blob/9....cpp#L1065
it would probably not hard to add, but I don't think adding another color space into the mix would just alienate more users,...
Instead of doing this it would be better to adjust vsDeoldify parameters to use RGB instead of HSV.
+
You are nearly using 50 parameters!
Instead of adding more stuff (nobody aside from you can/will use), better think about how to throw out stuff or at least define (tested) presets which allow to make the whole thing more approachable.
Cu Selur
Ps: I attached for you a modified vsViewer version.
Also, since the video usually isn't RGB, so not sure whether using the Display values is accurate:
When using the DisplayRGB values as base
https://github.com/Selur/vsViewer/blob/9....cpp#L1065
it would probably not hard to add, but I don't think adding another color space into the mix would just alienate more users,...
Instead of doing this it would be better to adjust vsDeoldify parameters to use RGB instead of HSV.
Code:
def ddeoldify_stabilizer(clip: vs.VideoNode, dark: bool = False, dark_p: list = [0.2, 0.8, "0:30,300:360"], smooth: bool = False, smooth_p: list = [0.3, 0.7, 0.9, 0.05, "none"], stab: bool = False, stab_p: list = [5, 'A', 1, 15, 0.2, 0.15, "300:360|0.5,0.0"], render_factor: int = 24) -> vs.VideoNode:
Code:
def ddeoldify(
clip: vs.VideoNode, method: int = 2, mweight: float = 0.5, deoldify_p: list = [0, 24, 1.0, 0.0], ddcolor_p: list = [1, 24, 1.0, 0.0, True], ddtweak: bool = False, ddtweak_p: list = [0.0, 1.0, 2.5, True, 0.2, 0.5, 1.5, 0.5, "300:360|0.3,0.2"], cmc_tresh: float = 0.2, lmm_p: list = [0.2, 0.8, 1.0], alm_p: list = [0.8, 1.0, 0.15], cmb_sw: bool = False, device_index: int = 0, torch_dir: str = model_dir) -> vs.VideoNode:
You are nearly using 50 parameters!
Instead of adding more stuff (nobody aside from you can/will use), better think about how to throw out stuff or at least define (tested) presets which allow to make the whole thing more approachable.
Cu Selur
Ps: I attached for you a modified vsViewer version.