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
Encoding folder update #8
Encoding folder update #8
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes.
There are quite a few issues about the content that even I as a non-encoder can spot to be wrong, so it will need a pass or two, but every change is an improvement at this point.
Left a few comments for consideration before I merge.
@@ -286,3 +286,5 @@ Just extract it somewhere. | |||
--- | |||
|
|||
[^1]: It should be noted that the author strongly disagrees with this sentiment. The two have a lot in common, and any capable Avisynth encoder could reach a similar leven in Vapoursynth within a few months, maybe even weeks. At least I'm honest, okay? | |||
|
|||
[^2]: https://github.com/Irrational-Encoding-Wizardry/guide.encode.moe/issues/1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of adding a link to this, we should just fix the text.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, but not really sure what the author of https://github.com/jpsdr/x264/releases meant. I didn't want to put something wrong there because I'm not sure exactly what happened on the release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, I guess something is better than nothing.
Also, good job with the separation of concern with these PRs 👍 |
Mostly just semantic linefeed updates, for future cleaning and adding. Also added blocky artifact pitcure and additional explanation.