Skip to content

Fix import statement for netcdf_file in calcam.py - #510

Merged
Mateasek merged 3 commits into
cherab:developmentfrom
munechika-koyo:fix/calcam
Jul 29, 2026
Merged

Fix import statement for netcdf_file in calcam.py#510
Mateasek merged 3 commits into
cherab:developmentfrom
munechika-koyo:fix/calcam

Conversation

@munechika-koyo

Copy link
Copy Markdown
Member

Summary

This is the change to the netcdf_file import statement due to the following warning message from scipy:

.pixi/envs/test/lib/python3.14/site-packages/cherab/tools/observers/calcam.py:21
  /Users/koyo/Documents/cherab/imas/.pixi/envs/test/lib/python3.14/site-packages/cherab/tools/observers/calcam.py:21: DeprecationWarning: Please import `netcdf_file` from the `scipy.io` namespace; the `scipy.io.netcdf` namespace is deprecated and will be removed in SciPy 2.0.0.
    from scipy.io.netcdf import netcdf_file

@Mateasek

Copy link
Copy Markdown
Member

Thanks for this @munechika-koyo

@Mateasek

Copy link
Copy Markdown
Member

I realised, could you update changelog please?

@munechika-koyo

Copy link
Copy Markdown
Member Author

I realised, could you update changelog please?

Sure! I updated it.

@Mateasek Mateasek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @Mateasek, Ill merge this into development.

@Mateasek
Mateasek merged commit c7fa05e into cherab:development Jul 29, 2026
6 checks passed
@munechika-koyo
munechika-koyo deleted the fix/calcam branch July 29, 2026 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants