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

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 25 10:42:18 PDT 2021


thopre added a comment.

In D111289#3084987 <https://reviews.llvm.org/D111289#3084987>, @asl wrote:

> @thopre I doubt the refactorings of this kind (change one plotting framework into another) could be split into smaller patches just by the nature of the change, you cannot have "half plot" and one essentially needs all the features ready for the review. So, such large patches are inevitable

I was afraid that'd be the case indeed. I'll try to give it a look but please bear with me as I'm not sure when I'll be able to set aside enough time to review this.


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