-
Notifications
You must be signed in to change notification settings - Fork 300
Replace pyke nopyke #4198
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Replace pyke nopyke #4198
Changes from 49 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
f427f85
First steps in parallel rules implementation.
pp-mo 1bb7338
Rename some things + add comments.
pp-mo 44bf952
Headers, tweaks, reorg test__load_cube.
pp-mo 5e3cb05
Fix handling of coord-systems.
pp-mo dfa6074
Remove extra cdl.
pp-mo 1aa4676
Add more grid-mapping tests; tidy testcode structure a bit.
pp-mo 87d3347
Test more grid types : rotated and some non-latlon (WIP).
pp-mo 88a1b2d
Fix 'checker' call usage.
pp-mo ad3ec2e
Reorganise testcode inheritance to provide testclasses for pyke/nonpy…
pp-mo b58c4bc
Tests for all supported grid-mappings. Note which rules trigger in ea…
pp-mo 04b079a
Disable testing against non-pyke code, for now.
pp-mo 1e0833e
Revert old tests/test_netcdf.py
pp-mo b119c7b
Odd clarifying comments.
pp-mo 364ea5b
Tidy testcase creation a bit.
pp-mo 4487961
Tests for mapping-types mismatch between coords+grid-mapping; Test no…
pp-mo 3f6e136
Refactor to put different test areas in their own sourcefiles.
pp-mo fe8ee7e
Small fix to actions code.
pp-mo 75550fc
Move grid-mapping-specific content out of common load_cube__activate …
pp-mo 97f2976
Add tests for time rules.
pp-mo 6a82183
Simplify and remove unused keys; check coord classes; test dim+aux sh…
pp-mo b59f0e7
Simpler 'Opts' implementation.
pp-mo 965ca8a
Tidy testing classes a bit.
pp-mo e3fd23d
Tests for hybrid vertical coords.
pp-mo 2a6cfd8
Small review changes.
pp-mo e776618
Regularise per-test records of rules triggered.
pp-mo 7f92f7b
Added tests for auxiliary lat+lon coords.
pp-mo 3c00044
Tests for remaining miscellaneous rules.
pp-mo 1cc4c7d
Review: fix typos.
pp-mo e50cdd5
Add testing option to compare pyke and nonpyke loads.
pp-mo 2bfeff6
Fixes to grid-mapping/dimcoord actions: passing all compare-tests.
pp-mo 72058b0
Actions for remaining 'miscellaneous' behaviour: All tests passing, i…
pp-mo f8a1bbe
Improved comments in actions routines.
pp-mo 6eaf756
Added actions for formulae (aka hybrid coords, factories).
pp-mo 09df997
Tiny fix.
pp-mo be9c0f1
Small review changes.
pp-mo 0c5a5f6
Added basic tests for Engine class.
pp-mo b55a925
Raise actual warning for unrecognised formulae.
pp-mo 0da7ef3
Remove unreachable warning code.
pp-mo 4bba400
Remove unreachable rule-fail code.
pp-mo eb46d51
Testcase for multiple hybrid coordinates.
pp-mo ce1e438
Fix rotated-aux-latitude test.
pp-mo 2fdcdf9
Isort fixes.
pp-mo f985b0c
Remove pyke and simplify testing.
pp-mo b0ac323
Ensure a definite ordering for formula-root processing.
pp-mo c26db97
Odd corrections to comments.
pp-mo 2f9d462
Updated all 'rules triggered' comments.
pp-mo c87fca8
Fix scope-bleed from actions test debugging, and turn it off.
pp-mo b40a446
Updated lockfiles.
pp-mo 9cb18fa
Small fixes to comments.
pp-mo a05f7b6
Remove refs to pyke in project config files.
pp-mo 794cab5
Review changes for comments: fix, clarify + make consistent.
pp-mo 97b20aa
Review change: correct test comment.
pp-mo 16be67b
Review changes: clarify comments.
pp-mo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -20,7 +20,6 @@ | |
| "scipy": [], | ||
|
|
||
| "setuptools": [], | ||
| "pyke": [], | ||
| "six": [], | ||
|
|
||
| "nose": [], | ||
|
|
||
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| # Copyright Iris contributors | ||
| # | ||
| # This file is part of Iris and is released under the LGPL license. | ||
| # See COPYING and COPYING.LESSER in the root of the repository for full | ||
| # licensing details. | ||
| """ | ||
| Support for cube-specific CF-to-Iris translation operations. | ||
|
|
||
| Interprets CF concepts identified by :mod:`iris.fileformats.cf` to add | ||
| components into loaded cubes. | ||
|
|
||
| For now : the API mimics :class:`pyke.knowledge_engine.engine`. | ||
| As this is aiming to replace the old Pyke-based logic rules. | ||
| TODO: simplify once the parallel operation with Pyke is no longer required. | ||
|
|
||
| """ | ||
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.