[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