Fix ax.scatter for atom-project band unfolding

QijingZheng / VaspBandUnfolding

A collection of python scripts that deal with VASP outpts, e.g. WAVECAR, POTCAR etc.

198 stars 89 forks source link

Fix ax.scatter for atom-project band unfolding #14

Closed kavanase closed 3 years ago

kavanase commented 3 years ago

Hi @QijingZheng! Thanks for developing a fantastic package!

Just a quick fix for the ax.scatter function call when doing atom-projected band unfolding (otherwise you end up an error saying s in scatter must be an integer or the same size as x or y).

Tested and fixes the issue.

QijingZheng commented 3 years ago

Hi @kavanase

Thank you for pointing out the error.

Sorry for the late reply. Somehow, github does not remind me of the pull request.

This error does not show up when I used matplotlib version 3.1. When I upgraded matplotlib today, this error pops up. I have fixed it and I also removed the k-point for-loop there in the scatter plot.

Qijing

kavanase commented 3 years ago

Hi @QijingZheng

That's great, thanks!

I'll close the PR now so