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

Adds docker support for all the executables #33

Merged
merged 2 commits into from
Oct 9, 2016

Conversation

leandromoreira
Copy link
Contributor

fixes #32

@li-zhi li-zhi merged commit 26fbaee into Netflix:master Oct 9, 2016
@li-zhi
Copy link
Contributor

li-zhi commented Oct 9, 2016

@leandromoreira I am also editing the README a bit.

@leandromoreira
Copy link
Contributor Author

That's fine, English is not my native language, also is there a way to know what is each model of the folder https://github.com/Netflix/vmaf/tree/master/resource/model ? (I mean what is trained model most of the people can use... [probably the one you trained with netflix+non-netflix videos] )

@li-zhi
Copy link
Contributor

li-zhi commented Oct 9, 2016

Please see
https://github.com/Netflix/vmaf/blob/master/python/core/quality_runner.py#L254
For VMAF v.0.3.2, the model should be nflxall_vmafv4.pkl. Apologize for not keeping the version numbers in sync. Will do that in the future. I will add this to README file also.

nilfm pushed a commit that referenced this pull request Dec 8, 2022
Merge in MCE/vmaf-private-lts from feature/add-placeholder-for-new-phone-vmaf-christosb to master

* commit '78344e34dda68668b3b39fa54147d77a2c66e8d1':
  Add placeholder for new phone VMAF.
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

Successfully merging this pull request may close these issues.

Adds Docker support for other executables
2 participants