Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

changing function name #2

Closed
realfinder opened this issue Jun 12, 2020 · 11 comments
Closed

changing function name #2

realfinder opened this issue Jun 12, 2020 · 11 comments

Comments

@realfinder
Copy link

since this plugin already has so many changes, isn't better to make it MPEG2DecPlus? and remove all disabled parameters

@Asd-g
Copy link
Owner

Asd-g commented Jun 13, 2020

Yes, it's probably better. LumaYUV could be removed too...

@realfinder
Copy link
Author

I don't use it, so yes, then it become pure MPEG2Dec

@realfinder
Copy link
Author

and maybe it better to move LumaYUV and BlindPP to DeBlock plugin

@realfinder
Copy link
Author

and btw, I think it should be named as MPEG2DecLite, since Plus mean with more things and that not true since it less that older one (even old MPEG2DecPlus is less than original one)

maybe MiniMPEG2Dec not MPEG2DecLite

@Asd-g
Copy link
Owner

Asd-g commented Jun 19, 2020

I made quick test and ColorYUV() is faster than LumaYUV() (both identical output).
Is BlindPP useful nowadays?

@realfinder
Copy link
Author

it used in some scripts like http://www.avisynth.nl/index.php/FunkyDeBlock

I think it's more or less useful as deblock plugin so it should be in it (deblock plugin)

@realfinder
Copy link
Author

and about the name, I have another suggestion, which is make it d2vsource as VS one (which I think is more suitable)

@Asd-g
Copy link
Owner

Asd-g commented Jun 19, 2020

I like that d2vsource name.
I would prefer to move BlindPP to a standalone plugin.

@realfinder
Copy link
Author

I have no problem with make it a standalone plugin

@Asd-g
Copy link
Owner

Asd-g commented Jun 21, 2020

I'll not bother with BlindPP.
FunkyDeBlock probably can use deblock and can get high support in addition.

@realfinder
Copy link
Author

ok, and since the rename is done I will close this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants