Added useful editions to animation plotting #8
Conversation
Added vertical and horizontal limits to animation, as well as the possibility of plotting flux surfaces and changing the cmap of animation.
xfeltor/plotting.py
Outdated
| @@ -267,16 +281,22 @@ def animate_pcolormesh( | |||
|
|
|||
| # Add title and axis labels | |||
| ax.set_title(variable) | |||
| if None not in (xmin, xmax): | |||
There was a problem hiding this comment.
You can pass None to ax.set_xlim:
ax.set_xlim(None, 10)Passing None leaves the limit unchanged. You could remove the if None not in (min, max) then.
|
It should be edited now. I am not sure why the commit appears like 8 hours ago when I did it right now. It might be something related with local times. |
Welcome to Codecov 🎉Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment. Thanks for integrating Codecov - We've got you covered ☂️ |
|
@rgerru Could you run black for code formatting on the files |
|
Hi Gregor, I have never used black or any tool to homogenize the formating of files. I fear it is something related with my code editor that changed some formating. Could you enforced it from your end? As it seems like you know what is failing. I have already checked indentations and they seem coherent with the rest of the code when opened with my editor, so it must be something related with how the editor opens and then saves the file back. |
|
I submitted a PR to |
Added vertical and horizontal limits to animation, as well as the possibility of plotting flux surfaces and changing the cmap of animation.