Remove unit and integration tests #21
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: Lockfile bot <[email protected]>
* Failing tests * Start with the tests * Draft implementation * Test results * Account for masked data and multiple axes * Testing, bug fixes * Reduce duplication * Update .cml and fix test failures * Don't use map_complete_blocks * Use dask versions of functions * Test and fix masked data handling * Use da.from_array instead of as_lazy_data * Test chunking, fix dask.array import * Restore _lazy_data.py as we're not using those changes
* First suggestion for weighted aggregated_by * Added lazy weighted aggregation * Added first tests for weighted aggregated_by * Added further tests and allowed returned kwarg * Added unit tests for weighted aggregated_by * Moved _get_aggregator_fn to iris.analyis * Reused 'returned' keyword * Added test for lazy output if 'returned' is used * Removed checksum=False from tests * Expanded test for lazy aggregation since PERCENTILE is lazy now * Added schlunma to common links
* Attempt faster add-aux-fact * Added whatsnew draft. * Put PR ref into whatsnew. * Review changes.
* Credit reviewer in what's new contributions. * Correct example link in what's new docs. Co-authored-by: Will Benfold <[email protected]> Co-authored-by: Will Benfold <[email protected]>
) * nans in cells satisfy equality * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * address review comments Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Netcdf saving: workaround known netcdf bug. * Fix netcdf dim renaming for anonymous dims. * Fixes to test-mesh generation to preserve save-load roundtrips. * Added tests for all the different places where code was fixed. * Tidy some formatting and comments. * Fix rebase error? * Review changes.
Co-authored-by: Lockfile bot <[email protected]>
* Support in helpers * coord_systems changes * More additions * Revert some FUTURE guards and condense datum application * Fixing tests and improving * More additions * Tests pass * transform_points test * Review fixes * In datums, unknown means None * Fix test * Suppress FutureWarning in the tests that were failing due to it * Review fixes * What's new * Extra test result
Bumps [peter-evans/create-pull-request](https://github.com/peter-evans/create-pull-request) from 4.0.2 to 4.0.3. - [Release notes](https://github.com/peter-evans/create-pull-request/releases) - [Commits](peter-evans/create-pull-request@bd72e1b...f094b77) --- updated-dependencies: - dependency-name: peter-evans/create-pull-request dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Lockfile bot <[email protected]>
* Replace nose with pytest * Reinstate magic processor number * add psutil, fstrings and rename TestMixin
* Caching * Move _optimum_chunksize docstring * Should be copying coords because they're mutable * Handle failure of datetime comparisons gracefully * Review fixes and more careful checks on mutable objects * coord_system doesn't have 'copy' but __repr__ covers the whole thing (I think) * Review fixes * Revert coord_system changes * Still need to handle tuple issue * What's new * Fix prompted by test failure * Limit cache size * Work around comparable datetime issue
* Initial moves to introduce result_coord_func * climatological keyword * Climatology argument to aggregated_by * Fix the tests * Review changes from RC * Test fixes * Further test fixes * Move test code back out after rebase * Different fix for self.coord_dims failure * Add ifmain * Revert inadvertent doctest changes * Review improvements * Fix doctest black broke * Stop new tests being executable, which makes the test runner miss them * Docstrings to comments so they don't obscure test names in results * Apply suggestions from code review Co-authored-by: Ruth Comer <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Review fixes * What's new Co-authored-by: Ruth Comer <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…ciTools#4739) * Mesh docs - include examples of converting different mesh formats. * SMC mesh example fix gotcha. * Add sample images to mesh conversion examples. * Mesh conversion examples - fix bullet points. * Mesh conversion examples - print meshes. * Whats New entry. * Mesh conversion examples - more detailed comments.
* update nemo example * clearer comment
* Extend any kwarg array * Add test * Add comments and update imagerepo.json * Try using the value as an array, in case it's been given as a list etc. * What's new
* Set stacklevel to get more useful FutureWarning * Remove spare printing * Only generate crs and globe on demand, handle attr setting * Updates docstrings to make it clearer what's going on * Additional tests (and fixes found from testing) * Warn that inverse_flattening setting doesn't help
Co-authored-by: Lockfile bot <[email protected]>
Co-authored-by: Lockfile bot <[email protected]>
* Permit fast percentile with mdtol=0 * What's new * Check arrays like they're masked, and make the expected arrays masked * Bigger arrays to avoid np quirk, bug fix, check masking in 2d_multi * Suppress numpy masked percentile warning * Use a larger test array that numpy handles how it handles most of them (the worse case) * Broadcast the mask in the non-lazy case with added dim * Bug fix on ndim test * Review fixes
* Correct heading for v3.4 release highlights. * Include patch number in release highlights heading. Co-authored-by: lbdreyer <[email protected]> Co-authored-by: lbdreyer <[email protected]>
updates: - [github.com/pycqa/isort: 5.10.1 → v5.11.3](PyCQA/isort@5.10.1...v5.11.3) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Bumps [actions/stale](https://github.com/actions/stale) from 6 to 7. - [Release notes](https://github.com/actions/stale/releases) - [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md) - [Commits](actions/stale@v6...v7) --- updated-dependencies: - dependency-name: actions/stale dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
updates: - [github.com/pycqa/isort: v5.11.3 → 5.11.4](PyCQA/isort@v5.11.3...5.11.4) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* unpin theme and set default style to light (not dark) * added whatsnew and fixed typo. * minor tweaks
* remove spelling config and docs, this didnt pan out. * added line break
* Fix link checks. * What's New entry. * ECMWF about experiment. * ECMWF apps experiment. * Remove ECMWF link. * More specific What's New entry.
* Updated environment lockfiles * Use set_ylim for test_2d_coord_bounds_platecarree. * Don't use np.float for maths tests. * Remove uses of np.int . Co-authored-by: Lockfile bot <[email protected]>
* Updated citation * Added WhatsNew * Reference version 3.4 in DOI Co-authored-by: Bouwe Andela <[email protected]> * Suggest updating the citation at every release Co-authored-by: Bouwe Andela <[email protected]>
* [pre-commit.ci] pre-commit autoupdate updates: - [github.com/asottile/blacken-docs: v1.12.1 → 1.13.0](adamchainz/blacken-docs@v1.12.1...1.13.0) * Remove explicit black version dependency from blacken-docs. Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Martin Yeo <[email protected]>
* Iris Xarray docs page. * Add links. * Xarray page styling. * What's New entry. * Minor docs fixes. * Overall experience section. * Xarray supports other plotting backends through external packages. Co-authored-by: Deepak Cherian <[email protected]> * Section on converting between Iris and Xarray. * Clearer language around laziness and multi-processing. * To-do note about dates and fill values. * Move iris_xarray page into a new Community section. * Language fixes from @bjlittle review. Co-authored-by: Deepak Cherian <[email protected]>
updates: - [github.com/pycqa/isort: 5.11.4 → 5.12.0](PyCQA/isort@5.11.4...5.12.0) Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* add readme #showyourstripes * use defacto url src * update whatsnew * licensed to distributed
* Fixing typo's in Gitwash. * Updating the whatsnew to implement requested changes
* [pre-commit.ci] pre-commit autoupdate updates: - [github.com/psf/black: 22.12.0 → 23.1.0](psf/black@22.12.0...23.1.0) * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Set up `iris.plugins` namespace package * Provide convenience function to use a plugin * Basic documentation for plugins * Add a hint about plugins when load fails * Expose `use_plugin` in `__all__` * Add example usage * Split plugin documentation to its own page * Document how to create a plugin * Link to documentation * Correct "plain" code block -> "text" * Whatsnew
Co-authored-by: Lockfile bot <[email protected]>
* Unpin netcdf4. * Temporarily enable GHA on this branch. * Temporarily enable GHA on this branch. * Temporarily enable GHA on this branch. * Experiment to disable wheel CI on forks. * Disable segfaulting routines. * More temporary changes to get CI passing. * More temporary changes to get CI passing. * Finessed segfault skipping. * Bring in changed from SciTools#5061. * Re-instate test_load_laea_grid. * Adaptations to get the tests passing. * Use typing.Mapping instead. * Get doctests passing. * CF only resolve non-url filenames. * Confirm thread safety fixes. * Remove dummy assignment. * Restored plot_nemo What's New entry. * _add_aux_factories temporarily release global lock. * Remove per-file locking. * Remove remaining test workarounds. * Remove remaining comments. * Correct use of CFReader context manager. * Refactor for easier future maintenance. * Rename netcdf _thread_safe, add header. * Full use of ThreadSafeAggregators. * Full use of ThreadSafeAggregators. * Remove remaining imports of NetCDF4. * Test to ensure netCDF4 is via _thread_safe module. * More refined netcdf._thread_safe classes. * _thread_safe docstrings. * Restore original NetCDF code where possible. * Revert changes to 2.3.rst. * Update lockfiles. * Additions to _thread_safe.py * Remove temporary CI shims. * New locking stategy for NetCDFDataProxy. * NetCDFDataProxy simpler use of netCDF4 lock. * Update lock files. * Go back to using a Threading Lock. * Remove superfluous pass commands in test_cf.py. * Rename _thread_safe to _thread_safe_nc. * Rename thread safe classes to be 'Wrappers'. * Better contained getattr and setattr pattern. * Explicitly name netCDF4 module in _thread_safe_nc docstring. * Better docstring for _ThreadSafeWrapper. * Better comment about THREAD_SAFE_FLAG. * list() wrapping within _GLOBAL_NETCDF4_LOCK, to account for generators. * More accurate thread_safe docstrings in netcdf.saver. * Split netcdf integration tests into multiple modules. * Tests for non-thread-safe NetCDF behaviour. * Docstring accuracy. * Correct use of dask config set (context manager). * Update dependencies. * Review - don't need the first-class import of iris.tests. * Better name for the loading test. * Better selection of data to load. * What's New entry. * Improve tests. * Update lock files. * Increase chunking on test_save. --------- Co-authored-by: Patrick Peglar <[email protected]>
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #21 +/- ##
=======================================
Coverage ? 62.06%
=======================================
Files ? 87
Lines ? 21915
Branches ? 4802
=======================================
Hits ? 13601
Misses ? 7217
Partials ? 1097 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
`v3.4.1` 'merge-back' - What's New changes only
* Add coverage testing * Update lockfile * Include __repr__ in coverage * use posargs for coverage flag instead of env var * update whatsnew to mentions posargs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚀 Pull Request
Description
Consult Iris pull request check list