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

Oasis Models producing Deprecated key warnings after keys are updated #55

Closed
Charlie-Farrar opened this issue Aug 4, 2023 · 0 comments · Fixed by #84
Closed

Oasis Models producing Deprecated key warnings after keys are updated #55

Charlie-Farrar opened this issue Aug 4, 2023 · 0 comments · Fixed by #84
Assignees
Labels
bug Something isn't working
Milestone

Comments

@Charlie-Farrar
Copy link

Issue Description

When running the Oasis Models, they give back a Deprecated key in analysis_settings.json warning for the keys model_supplier_id and model_name_id :

Deprecated key in analysis_settings.json, "module_supplier_id" updated to "model_supplier_id"
Deprecated key in analysis_settings.json, "model_version_id" updated to "model_name_id"

Both of these keys have been update in analysis_settings.json, however, this warning persists.

This appears to be originating from:

self.logger.warning(f'Deprecated key in {self.settings_type}.json, "{old_key}" updated to "{new_key}"')
:

and

def __init__(self, schema=None, json_path=None):
self.SCHEMA_FILE = 'analysis_settings_schema.json'
self.compatibility_profile = [
CompatibilityMap(keys='module_supplier_id', updated='model_supplier_id', ver='1.23.0'),
CompatibilityMap(keys='model_version_id', updated='model_name_id', ver='1.23.0'),
]
super(AnalysisSettingSchema, self).__init__(schema, json_path, 'analysis_settings')
:

Steps to Reproduce (Bugs only)

  1. Pull OasisModels repo
  2. Navigate to {any model}/tests/test_x and run oasislmf model run --config oasislmf.json
  3. This will produce the warnings

Version / Environment information

1.28

@Charlie-Farrar Charlie-Farrar added bug Something isn't working invalid This doesn't seem right labels Aug 4, 2023
@Charlie-Farrar Charlie-Farrar changed the title Oasis Models producing 'Deprecated key' warnings after keys are updated Oasis Models producing Deprecated key warnings after keys are updated Aug 4, 2023
@sambles sambles removed the invalid This doesn't seem right label Jan 11, 2024
@sambles sambles linked a pull request Jan 11, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this from Todo to Done in Oasis Dev Team Tasks Jan 29, 2024
@awsbuild awsbuild added this to the 3.2.0 milestone Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants