Conversation
This change passes the encoding results from the generation side to the display side as JSON files. All usage of gviz code on the generation side is no longer needed. Note: This is a transition CL - the generation side will be able to generate both the old way and the new way until the new way works better than the old way.
Changed the overshoot rule in cpu/rate to be 0.1 dB per percent overshoot, rather than 0.1 dB per kbit/sec overshoot, which increased sanity of VP9 displays. Added in display of command lines as part of the results shown. Added in generation of new-style links in the index pages.
@louquillio can you look at the style application functions and Javascript? |
# See the License for the specific language governing permissions and | ||
# limitations under the License. | ||
# | ||
# A JSON-generating codec comparision tool. |
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.
Nit: comparison
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.
Done (full sweep).
No support for the theory that "comparision" is a valid spelling found. Also fixing one pylint error.
PTAL |
@@ -440,6 +445,10 @@ def Execute(self, bitrate, videofile, workdir): | |||
return self.context.codec.Execute( | |||
self.parameters, bitrate, videofile, workdir) | |||
|
|||
def EncodeCommandLine(self, bitrate, videofile, workdir): | |||
return self.context.codec.EncodeCommandLine( | |||
self.parameters, bitrate, videofile, workdir) |
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.
Nit: should indent two more when breaking in an invocation
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.
Done.
lgtm for python parts |
a887e76
to
0920c19
Compare
Redesigning the interface between the website parts and the Python backend parts to interchange JSON files instead of generating HTML files.
This makes for a simpler system, where display is handled on the display side, and only data is handled on the backend side.