Skip to content
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

Update VarHandler._get_table_freq() once all legacy land-atm handlers are refactored #219

Open
tomvothecoder opened this issue Sep 26, 2023 · 0 comments

Comments

@tomvothecoder
Copy link
Collaborator

def _update_table_ref(self, freq: str, realm: str, cmip_tables_path: str):
"""
Update the referenced CMIP table for cmorizing based on the selected
frequency and realm.
Parameters
----------
freq : str
The frequency for CMORizing, which is used to derive the appropriate
CMIP6 table.
realm : str
The realm.
cmip_tables_path : str
The path to the CMIP6 tables.
"""
# TODO: Implement logic from
# `e3sm_to_cmip.util._get_table_for_non_monthly_freq()`.
self.table = _get_table_for_non_monthly_freq(
self.name, self.table, freq, realm, cmip_tables_path
)

  • Move logic from _get_table_for_non_monthly_freq() this this method
  • Update downstream calls from old function to this method.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant