Add field option to plot_iso

[Laetitia1402]

No description provided.

[amorison]

Thank you for this PR. The change itself looks good. Could you please update the docstring so that the new option is documented? Copying the bit about the field argument in the docstring of plot_scalar would work well.

Also, this is more minor but could you fix the indentation and the trailing space that flake8 warns about? You can look at the results from the check, but it might be more convenient for you to run it locally with tox -e flake8 (tox can be installed via pip). Let me know if you run into troubles, I can fix the formatting when merging if you prefer.

[Laetitia1402]

Salut Adrien, Bonne année ! Je suis désolée je n'ai pas pu updater mon PR avant, il y a eu des cas de Covid au labo avant les vacances et je n'ai pas pu récupérer mon PC de boulot, je n'avais pas linux sur le mien. Pour faire ces modifs, je les fais en local et je dois refaire un PR ensuite du coup ? Laëtitia Le mar. 14 déc. 2021 à 15:27, Adrien Morison ***@***.***> a écrit :

…

Thank you for this PR. The change itself looks good. Could you please update the docstring so that the new option is documented? Copying the bit about the field argument in the docstring of plot_scalar would work well. Also, this is more minor but could you fix the indentation and the trailing space that flake8 warns about? You can look at the results from the check, but it might be more convenient for you to run it locally with tox -e flake8 (tox can be installed via pip). Let me know if you run into troubles, I can fix the formatting when merging if you prefer. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#82 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQRPNCJVXZCRBSSG5KZEFLDUQ5H3TANCNFSM5KAXULFA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[amorison]

Thanks for the corrections.

The type error pointed out by mypy in the CI is unrelated to those changes, and CI needs to be fixed anyway (see #83). Since the changes in this PR are simple, I'll merge it anyway.