vsavchenko added inline comments.
================ Comment at: clang/utils/analyzer/SATestBenchmark.py:139-140 + import matplotlib + import seaborn as sns + from matplotlib import pyplot as plt + ---------------- NoQ wrote: > 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. I agree with that, but python world has a few very widespread shortenings: ``` numpy as np matplotlib.pyplot as plt seaborn as sns pandas as pd ``` All the tutorials, official documentation and so on include these. So they might be even more recognizable as shortenings. 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
