[PATCH] D111289: [LNT] Refactored the Graph page to use the library plotly instead of jQuery.flot

Chris Matthews via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 25 12:20:48 PDT 2021


cmatthews added a comment.

As a start, split the non-patch related changes out. Specifically, at least split the changes in: test_matrix_page.py, test_api.py, the renames, the error message updates in views.py, the refactoring in api.py.

We need test coverage in all this code being added as well.

Move some of this out into a series of refactorings, before the main final change, that will help make it simpler to reason about the changes.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111289/new/

https://reviews.llvm.org/D111289



More information about the llvm-commits mailing list