Attempt: fix holoviews - xarray bug

[lang-m]

• Test with Python 3.8 and 3.10
• Fix holoviews vectorplot bug; this was actually a problem with xarray and numpy ufuncs when computing the inplane-angle using np.arctan2
• Hide FutureWarning and UserWarning from HoloViews

[github-actions[bot]]

PR Analysis

• 🎯 Main theme: The PR attempts to fix a bug related to holoviews and xarray by updating the Python version in the GitHub workflow and environment.

• 📝 PR summary: This PR updates the Python version in the GitHub workflow and environment files, adding support for Python 3.10. It also removes the fixed Python version from the environment file, allowing the workflow to test on multiple Python versions.

• 📌 Type of PR: Bug fix

• 🧪 Relevant tests added: No

• ⏱️ Estimated effort to review [1-5]: 1 This PR is relatively straightforward, involving changes to the Python version in the GitHub workflow and environment files. It should be quick and easy to review.

• 🔒 Security concerns: No

  PR Feedback

• 💡 General suggestions: The changes in this PR are clear and straightforward. However, it would be beneficial to add tests to ensure that the bug fix works as expected across different Python versions.

• 🤖 Code feedback:

  • relevant file: .github/workflows/workflow.yml suggestion: Consider adding a step in the workflow to print the Python version. This will help in debugging if any issues arise related to the Python version. [medium] relevant line: python-version: ${{ matrix.python-version }}

How to use

To invoke the PR-Agent, add a comment using one of the following commands: /review [-i]: Request a review of your Pull Request. For an incremental review, which only considers changes since the last review, include the '-i' option. /describe: Modify the PR title and description based on the contents of the PR. /improve [--extended]: Suggest improvements to the code in the PR. Extended mode employs several calls, and provides a more thorough feedback. /ask \<QUESTION>: Pose a question about the PR. /update_changelog: Update the changelog based on the PR's contents.

To edit any configuration parameter from configuration.toml, add --config_path=new_value For example: /review --pr_reviewer.extra_instructions="focus on the file: ..." To list the possible configuration parameters, use the /config command.

[lang-m]

@samjrholt I have disabled pre-commits for discretisedfield. I hope that maybe when we re-activate it later (after merging) the wrong error might disappear.

[codecov-commenter]

Codecov Report

All modified lines are covered by tests :white_check_mark:

Comparison is base (720a457) 93.12% compared to head (81a0e98) 93.16%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #494 +/- ## ========================================== + Coverage 93.12% 93.16% +0.03% ========================================== Files 28 28 Lines 3026 3027 +1 ========================================== + Hits 2818 2820 +2 + Misses 208 207 -1 ``` | [Files](https://app.codecov.io/gh/ubermag/discretisedfield/pull/494?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ubermag) | Coverage Δ | | |---|---|---| | [discretisedfield/plotting/hv.py](https://app.codecov.io/gh/ubermag/discretisedfield/pull/494?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ubermag#diff-ZGlzY3JldGlzZWRmaWVsZC9wbG90dGluZy9odi5weQ==) | `92.77% <100.00%> (+0.04%)` | :arrow_up: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/ubermag/discretisedfield/pull/494/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ubermag)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.