Skip to content

68 new env - #162

Merged
Josh Rackham (jrackham-mo) merged 9 commits into
MetOffice:environment-update-feature-branchfrom
mo-tgeddes:68_new_env
Sep 2, 2026
Merged

68 new env#162
Josh Rackham (jrackham-mo) merged 9 commits into
MetOffice:environment-update-feature-branchfrom
mo-tgeddes:68_new_env

Conversation

@mo-tgeddes

Copy link
Copy Markdown
Collaborator

Related to #68

Work so far on new environment upgrade

@mo-tgeddes

Copy link
Copy Markdown
Collaborator Author

Dask changes: iris/_lazy_data.py now gives an erros if meta is passed in as None (the default if not specified): ValueError: For performance reasons, metacannot beNoneifdata is anything other than a Numpy or Dask array.

This uses pandas dataframes rather than arrays, but shows that meta should be an empty structure of the same shape and type. The iris code also shows that.

@mo-tgeddes

Copy link
Copy Markdown
Collaborator Author

np.sctypeDict changes - I can't find anything in the documentation, however, previously results of np.sctypeDict would be a dictionary with values mapped to a number, i.e. {0: <class 'numpy.bool_'>,...}, with the environment upgrade it is now a dictionary of datatypes mapped to a name, i.e. {'bool': <class 'numpy.bool'>}.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There are a few failing unit tests, is this expected? Presumably they'll be fixed in other issues?

======================================================================== short test summary info =========================================================================
FAILED lib/ants/tests/command_parse/test_integration.py::TestAntsArgParser::test_missing_lbm - AssertionError: Expected 'exit' to be called once. Called 2 times.
FAILED lib/ants/tests/fileformats/test_integration.py::TestExceptions::test_no_datum_warning - Failed: Unexpected success
FAILED lib/ants/tests/utils/test_transform_bbox.py::TestDiffCS::test_point_lie_beyond_crs_definition - AssertionError: ValueError not raised
========================================================= 3 failed, 1245 passed, 2 skipped, 10 xfailed in 25.75s =========================================================

Comment thread lib/ants/fileformats/raster.py Outdated
Comment thread environment.yml
Co-authored-by: Josh Rackham <144251043+jrackham-mo@users.noreply.github.com>
Signed-off-by: Theo Geddes <theo.geddes@metoffice.gov.uk>
@mo-tgeddes

Copy link
Copy Markdown
Collaborator Author

Yes there are separate issues for the unit tests on the project board

@jrackham-mo
Josh Rackham (jrackham-mo) merged commit 45f0760 into MetOffice:environment-update-feature-branch Sep 2, 2026
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