-
Notifications
You must be signed in to change notification settings - Fork 50
Enable interpolation of time-varying input data with temporal controller #311
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
Open
Ed Hone (EdHone)
wants to merge
10
commits into
MetOffice:main
Choose a base branch
from
EdHone:242-temporal-interpolation
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 9 commits
Commits
Show all changes
10 commits
Select commit
Hold shift + click to select a range
2009352
Add interp code with original functionality still working
EdHone f350d30
Add duration enums
EdHone 24d0b48
Add enum parser and test
EdHone 125c8ec
Close to finished implementation
EdHone ed9481d
Working tests
EdHone 9ac56e9
Temporal interpolation test that runs and looks correct
EdHone 0844a96
Non sync test working and implementation fixed
EdHone 4e23897
Fix test syntax
EdHone 77de448
Self code review
EdHone 3e553c9
Merge branch 'main' into 242-temporal-interpolation
mo-lucy-gordon 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
94 changes: 94 additions & 0 deletions
94
components/lfric-xios/integration-test/lfric_xios_temporal_interp_test.f90
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,94 @@ | ||
| !----------------------------------------------------------------------------- | ||
| ! (C) Crown copyright 2026 Met Office. All rights reserved. | ||
| ! The file LICENCE, distributed with this code, contains details of the terms | ||
| ! under which the code may be used. | ||
| !----------------------------------------------------------------------------- | ||
|
|
||
| ! Tests the LFRic-XIOS temporal reading functionality | ||
| ! | ||
| program lfric_xios_temporal_interp_test | ||
|
|
||
| use constants_mod, only: i_timestep, r_second | ||
| use event_mod, only: event_action | ||
| use event_actor_mod, only: event_actor_type | ||
| use field_mod, only: field_type, field_proxy_type | ||
| use file_mod, only: FILE_MODE_READ, FILE_MODE_WRITE | ||
| use io_context_mod, only: callback_clock_arg | ||
| use lfric_xios_action_mod, only: advance | ||
| use lfric_xios_context_mod, only: lfric_xios_context_type | ||
| use lfric_xios_driver_mod, only: lfric_xios_initialise, lfric_xios_finalise | ||
| use lfric_xios_file_mod, only: lfric_xios_file_type, OPERATION_TIMESERIES | ||
| use lfric_xios_constants_mod, only: lx_second | ||
| use linked_list_mod, only: linked_list_type | ||
| use log_mod, only: log_event, log_level_info | ||
| use test_db_mod, only: test_db_type | ||
| use xios, only: xios_date | ||
|
|
||
| implicit none | ||
|
|
||
| type(test_db_type) :: test_db | ||
| type(lfric_xios_context_type), target, allocatable :: io_context | ||
|
|
||
| procedure(callback_clock_arg), pointer :: before_close | ||
| type(linked_list_type), pointer :: file_list | ||
| class(event_actor_type), pointer :: context_actor | ||
| procedure(event_action), pointer :: context_advance | ||
| type(field_type), pointer :: rfield | ||
| type(field_proxy_type) :: rproxy | ||
| type(xios_date) :: date | ||
| integer(i_timestep) :: file_freq | ||
|
|
||
| call test_db%initialise() | ||
| call lfric_xios_initialise( "test", test_db%comm, .false. ) | ||
|
|
||
| ! =============================== Start test ================================ | ||
|
|
||
| allocate(io_context) | ||
| call io_context%initialise( "test_io_context", 1, 10 ) | ||
|
|
||
| ! Fixed attribute of input data | ||
| file_freq = int(60.0_r_second / test_db%clock%get_seconds_per_step(), i_timestep) | ||
|
|
||
| file_list => io_context%get_filelist() | ||
| call file_list%insert_item( lfric_xios_file_type( "lfric_xios_interp_input", & | ||
| xios_id="lfric_xios_interp_input", & | ||
| io_mode=FILE_MODE_READ, & | ||
| operation=OPERATION_TIMESERIES, & | ||
| freq=file_freq, & | ||
| cyclic=.true., & | ||
| update_freq=20*lx_second, & | ||
| fields_in_file=test_db%temporal_fields ) ) | ||
| call file_list%insert_item( lfric_xios_file_type( "lfric_xios_interp_output", & | ||
| xios_id="lfric_xios_interp_output", & | ||
| io_mode=FILE_MODE_WRITE, & | ||
| operation=OPERATION_TIMESERIES, & | ||
| freq=1, & | ||
| fields_in_file=test_db%temporal_fields ) ) | ||
|
|
||
| before_close => null() | ||
| call io_context%initialise_xios_context( test_db%comm, & | ||
| test_db%chi, test_db%panel_id, & | ||
| test_db%clock, test_db%calendar, & | ||
| before_close ) | ||
|
|
||
|
|
||
| context_advance => advance | ||
| context_actor => io_context | ||
| call test_db%clock%add_event( context_advance, context_actor ) | ||
| call io_context%set_active(.true.) | ||
|
|
||
| do while (test_db%clock%tick()) | ||
| call test_db%temporal_fields%get_field("temporal_field", rfield) | ||
| rproxy = rfield%get_proxy() | ||
| call log_event("Valid data for this TS:", log_level_info) | ||
| print*,rproxy%data(1) | ||
| end do | ||
|
|
||
| deallocate(io_context) | ||
|
|
||
| ! ============================== Finish test ================================= | ||
|
|
||
| call lfric_xios_finalise() | ||
| call test_db%finalise() | ||
|
|
||
| end program lfric_xios_temporal_interp_test | ||
96 changes: 96 additions & 0 deletions
96
components/lfric-xios/integration-test/lfric_xios_temporal_interp_test.py
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,96 @@ | ||||||
| #!/usr/bin/env python3 | ||||||
| ############################################################################## | ||||||
| # (C) Crown copyright 2024 Met Office. All rights reserved. | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| # The file LICENCE, distributed with this code, contains details of the terms | ||||||
| # under which the code may be used. | ||||||
| ############################################################################## | ||||||
| """ | ||||||
| A set of tests which exercise the temporal reading functionality provided by | ||||||
| the LFRic-XIOS component. | ||||||
| The tests cover the reading of a piece of non-cyclic temporal data with data | ||||||
| points ranging from 15:01 to 15:10 in 10 1-minute intervals. The model start | ||||||
| time is changed to change how the model interacts with the data. | ||||||
| """ | ||||||
| from testframework import TestEngine, TestFailed | ||||||
| from xiostest import LFRicXiosTest | ||||||
| from pathlib import Path | ||||||
| import sys | ||||||
|
|
||||||
| ############################################################################### | ||||||
| class LfricXiosFullInterpTest(LFRicXiosTest): # pylint: disable=too-few-public-methods | ||||||
| """ | ||||||
| Tests the LFRic-XIOS temporal reading functionality for a full set of non-cyclic data | ||||||
| """ | ||||||
|
|
||||||
| def __init__(self): | ||||||
| super().__init__(command=[sys.argv[1], "cyclic_high_freq.nml"], processes=1) | ||||||
| self.gen_data('temporal_data.cdl', 'lfric_xios_interp_input.nc') | ||||||
| self.gen_data('cyclic_high_freq_kgo.cdl', 'cyclic_high_freq_kgo.nc') | ||||||
| self.gen_config( "cyclic_base.nml", "cyclic_high_freq.nml", | ||||||
| {"dt":10.0, | ||||||
| "timestep_end":'150'} ) | ||||||
|
|
||||||
| def test(self, returncode: int, out: str, err: str): | ||||||
| """ | ||||||
| Test the output of the interpolation test | ||||||
| """ | ||||||
|
|
||||||
| if returncode != 0: | ||||||
| print(out) | ||||||
| raise TestFailed(f"Unexpected failure of test executable: {returncode}\n" + | ||||||
| f"stderr:\n" + | ||||||
| f"{err}") | ||||||
|
|
||||||
| self.plot_output(Path(self.test_working_dir, 'lfric_xios_interp_input.nc'), | ||||||
| Path(self.test_working_dir, 'lfric_xios_interp_output.nc'), | ||||||
| 'temporal_field') | ||||||
|
|
||||||
| if not self.nc_data_match(Path(self.test_working_dir, 'lfric_xios_interp_input.nc'), | ||||||
| Path(self.test_working_dir, 'lfric_xios_interp_output.nc'), | ||||||
| 'temporal_field'): | ||||||
| raise TestFailed("Output data does not match input data for same time values") | ||||||
|
|
||||||
| return "Reading and interpolating data okay..." | ||||||
|
|
||||||
| class LfricXiosNonSyncInterpTest(LFRicXiosTest): # pylint: disable=too-few-public-methods | ||||||
| """ | ||||||
| Tests the LFRic-XIOS temporal reading functionality for a full set of non-cyclic data | ||||||
| """ | ||||||
|
|
||||||
| def __init__(self): | ||||||
| super().__init__(command=[sys.argv[1], "cyclic_high_freq.nml"], processes=1) | ||||||
| self.gen_data('temporal_data.cdl', 'lfric_xios_interp_input.nc') | ||||||
| self.gen_data('cyclic_high_freq_kgo.cdl', 'cyclic_high_freq_kgo.nc') | ||||||
| self.gen_config( "cyclic_base.nml", "cyclic_high_freq.nml", | ||||||
| {"dt":10.0, | ||||||
| "calendar_start":"2024-01-01 15:03:20", | ||||||
| "timestep_end":'30'} ) | ||||||
|
|
||||||
| def test(self, returncode: int, out: str, err: str): | ||||||
| """ | ||||||
| Test the output of the interpolation test | ||||||
| """ | ||||||
|
|
||||||
| if returncode != 0: | ||||||
| print(out) | ||||||
| raise TestFailed(f"Unexpected failure of test executable: {returncode}\n" + | ||||||
| f"stderr:\n" + | ||||||
| f"{err}") | ||||||
|
|
||||||
| self.plot_output(Path(self.test_working_dir, 'lfric_xios_interp_input.nc'), | ||||||
| Path(self.test_working_dir, 'lfric_xios_interp_output.nc'), | ||||||
| 'temporal_field') | ||||||
|
|
||||||
| if not self.nc_data_match(Path(self.test_working_dir, 'lfric_xios_interp_input.nc'), | ||||||
| Path(self.test_working_dir, 'lfric_xios_interp_output.nc'), | ||||||
| 'temporal_field'): | ||||||
| raise TestFailed("Output data does not match input data for same time values") | ||||||
|
|
||||||
| return "Reading and interpolating data okay..." | ||||||
|
|
||||||
|
|
||||||
|
|
||||||
| ############################################################################## | ||||||
| if __name__ == "__main__": | ||||||
| TestEngine.run(LfricXiosFullInterpTest()) | ||||||
| TestEngine.run(LfricXiosNonSyncInterpTest()) | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I ran flake8 on this file and there's a fair few lint errors so please could you remove those |
||||||
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.