adareau / HAL

HAL Atom Locator
GNU General Public License v3.0
2 stars 1 forks source link

Feat/quick density plot #110

Closed quentinmarolleau closed 3 years ago

quentinmarolleau commented 3 years ago

e.g. with fake data: image

adareau commented 3 years ago

OK, looks nice. I'm just curious: do you really have to use pandas et seaborn, rather than numpy/matplotlib ? The point is just to avoid having multiple packages & standards across HAL to handle data & plots...

quentinmarolleau commented 3 years ago

OK, looks nice. I'm just curious: do you really have to use pandas et seaborn, rather than numpy/matplotlib ? The point is just to avoid having multiple packages & standards across HAL to handle data & plots...

probably not, but it is MUCH simpler to manipulate dataframes to do this kind of stuff (especially with the groupby method of pd.DataFrames). Btw pip install seaborn will also install pandas.

Additionaly, the plot is ultimately handled with matplotlib since seaborn is just a kind of add-on that makes the visualisations of pandas dataframes easier, and is then piped into a matplotlib figure/axes.