[PATCH] D83539: [analyzer][tests] Introduce analyzer benchmarking framework

[Valeriy Savchenko via Phabricator via cfe-commits]

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