NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land.
Box-and-whisker diagrams 👍 ================ Comment at: clang/utils/analyzer/SATestBenchmark.py:18 + +import pandas as pd +from typing import List, Tuple ---------------- @Szelethus ??? ================ Comment at: clang/utils/analyzer/SATestBenchmark.py:139-140 + import matplotlib + import seaborn as sns + from matplotlib import pyplot as plt + ---------------- Why shorten? It's not that we're saving a lot of typing, and i think it's easier to read when package names are recognizable. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83539/new/ https://reviews.llvm.org/D83539 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
