improve documentation #912
Conversation
| ### ADM | ||
|
|
||
| ADM was previously named Detail Loss Metric (DLM), described in S. Li, F. Zhang, L. Ma, and K. Ngan, “Image Quality Assessment by Separately Evaluating Detail Losses and Additive Impairments,” IEEE Transactions on Multimedia, vol. 13, no. 5, pp. 935–949, Oct. 2011. | ||
|
|
||
| DLM is an image quality metric based on the rationale of separately measuring the loss of details which affects the content visibility, and the redundant impairment which distracts viewer attention. The original metric combines both DLM and additive impairment measure (AIM) to yield a final score. In VMAF, only the DLM part is added as an elementary metric. Particular care was taken for special cases, such as black frames, where numerical calculations for the original formulation break down. |
There was a problem hiding this comment.
I wasn't too sure about this one, and I'm missing some details here (no pun intended). Please add accordingly.
PS: I took the liberty of quoting from the blog post here. If this is an issue, I can rephrase.
There was a problem hiding this comment.
:)
This is actually pretty accurate. Thanks for the great effort. I'll take a detailed look and the rest of the modifications.
There was a problem hiding this comment.
Well it's a verbatim copy from the Netflix blog post. Hence my PS — but if it's fine for you, it's fine for me!
There was a problem hiding this comment.
ADM was a historical name for DLM and AIM combined.
There was a problem hiding this comment.
Yes.
We probably should clean up the codebase to avoid the confusion. Added this to my todo list.
There was a problem hiding this comment.
I guess a better description in the documentation would be enough. If you change the name of the metric it might cause problems or confusion down the road, since people will already be used to it, and there might be scripts parsing the name of the indicator etc.
There was a problem hiding this comment.
Fair point. Let me think about this further.
- Create an index page for all documentation pages - Clean up README.md to reference documentation overview, from where people can find all docs - Harmonize Markdown headings in all individual pages - Add dedicated page for core metrics - Add NEG to model page - Harmonize naming of Markdown documents - Move PDFs to separate directories - Fix a few spelling errors, broken HTTP(S) links etc.
slhck
force-pushed
the
improve-documentation
branch
from
82d2285 to
609aa8e
Compare
This somewhat fixes #125