Skip to content

Commit c05eed8

Browse files
authored
Fix for compare suite and rename (#644)
* minor fixes for comparison tests * add comparison log type to task questions * rename compare suite
1 parent 6ce61a6 commit c05eed8

File tree

7 files changed

+30
-12
lines changed

7 files changed

+30
-12
lines changed

src/swell/suites/compare_variational/eva/comparison_increment-geos_atmosphere.yaml renamed to src/swell/suites/compare/eva/comparison_increment-geos_atmosphere.yaml

File renamed without changes.

src/swell/suites/compare_variational/eva/comparison_increment-geos_marine.yaml renamed to src/swell/suites/compare/eva/comparison_increment-geos_marine.yaml

File renamed without changes.

src/swell/suites/compare_variational/eva/comparison_jedi_log-geos_atmosphere.yaml renamed to src/swell/suites/compare/eva/comparison_jedi_log-geos_atmosphere.yaml

File renamed without changes.

src/swell/suites/compare_variational/eva/comparison_jedi_log-geos_marine.yaml renamed to src/swell/suites/compare/eva/comparison_jedi_log-geos_marine.yaml

File renamed without changes.
File renamed without changes.

src/swell/suites/compare_variational/suite_config.py renamed to src/swell/suites/compare/suite_config.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ class SuiteConfig(QuestionContainer, Enum):
2020

2121
# --------------------------------------------------------------------------------------------------
2222

23-
compare_variational = QuestionList(
23+
compare = QuestionList(
2424
list_name="compare",
2525
questions=[
2626
sq.all_suites,
@@ -38,7 +38,7 @@ class SuiteConfig(QuestionContainer, Enum):
3838
compare_variational_marine = QuestionList(
3939
list_name="compare_variational_marine",
4040
questions=[
41-
compare_variational,
41+
compare,
4242
qd.comparison_log_type('variational'),
4343
qd.model_components(['geos_marine']),
4444
]
@@ -49,7 +49,7 @@ class SuiteConfig(QuestionContainer, Enum):
4949
compare_variational_atmosphere = QuestionList(
5050
list_name="compare_variational_atmosphere",
5151
questions=[
52-
compare_variational,
52+
compare,
5353
qd.comparison_log_type('variational'),
5454
qd.model_components(['geos_atmosphere']),
5555
]
@@ -60,7 +60,7 @@ class SuiteConfig(QuestionContainer, Enum):
6060
compare_fgat_marine = QuestionList(
6161
list_name="compare_fgat_marine",
6262
questions=[
63-
compare_variational,
63+
compare,
6464
qd.comparison_log_type('fgat'),
6565
qd.model_components(['geos_marine']),
6666
]

src/swell/tasks/task_questions.py

Lines changed: 26 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -451,6 +451,15 @@ class TaskQuestions(QuestionContainer, Enum):
451451

452452
# --------------------------------------------------------------------------------------------------
453453

454+
JediOopsLogParser = QuestionList(
455+
list_name="JediOopsLogParser",
456+
questions=[
457+
qd.comparison_log_type(),
458+
]
459+
)
460+
461+
# --------------------------------------------------------------------------------------------------
462+
454463
LinkGeosOutput = QuestionList(
455464
list_name="LinkGeosOutput",
456465
questions=[
@@ -517,7 +526,8 @@ class TaskQuestions(QuestionContainer, Enum):
517526
qd.observations(),
518527
qd.total_processors(),
519528
qd.window_offset(),
520-
qd.window_type()
529+
qd.window_type(),
530+
qd.comparison_log_type('convert_state_soca2cice'),
521531
]
522532
)
523533

@@ -534,6 +544,7 @@ class TaskQuestions(QuestionContainer, Enum):
534544
qd.jedi_forecast_model(),
535545
qd.observations(),
536546
qd.observing_system_records_path(),
547+
qd.comparison_log_type('ensmeanvariance'),
537548
]
538549
)
539550

@@ -543,7 +554,8 @@ class TaskQuestions(QuestionContainer, Enum):
543554
list_name="RunJediFgatExecutable",
544555
questions=[
545556
run_jedi_executable,
546-
qd.marine_models()
557+
qd.marine_models(),
558+
qd.comparison_log_type('fgat')
547559
]
548560
)
549561

@@ -561,7 +573,8 @@ class TaskQuestions(QuestionContainer, Enum):
561573
qd.ensemble_num_members(),
562574
qd.generate_yaml_and_exit(),
563575
qd.jedi_forecast_model(),
564-
qd.total_processors()
576+
qd.total_processors(),
577+
qd.comparison_log_type('hofx')
565578
]
566579
)
567580

@@ -577,7 +590,8 @@ class TaskQuestions(QuestionContainer, Enum):
577590
qd.generate_yaml_and_exit(),
578591
qd.jedi_forecast_model(),
579592
qd.save_geovals(),
580-
qd.total_processors()
593+
qd.total_processors(),
594+
qd.comparison_log_type('ensemblehofx'),
581595
]
582596
)
583597

@@ -616,7 +630,8 @@ class TaskQuestions(QuestionContainer, Enum):
616630
qd.vertical_localization_ioda_vertical_coord_group(),
617631
qd.vertical_localization_lengthscale(),
618632
qd.vertical_localization_method(),
619-
qd.perhost()
633+
qd.perhost(),
634+
qd.comparison_log_type('localensembleda'),
620635
]
621636
)
622637

@@ -633,7 +648,8 @@ class TaskQuestions(QuestionContainer, Enum):
633648
qd.jedi_forecast_model(),
634649
qd.observing_system_records_path(),
635650
qd.total_processors(),
636-
qd.obs_thinning_rej_fraction()
651+
qd.obs_thinning_rej_fraction(),
652+
qd.comparison_log_type('obsfilters')
637653
]
638654
)
639655

@@ -646,7 +662,8 @@ class TaskQuestions(QuestionContainer, Enum):
646662
qd.generate_yaml_and_exit(),
647663
qd.single_observations(),
648664
qd.window_length(),
649-
qd.window_offset()
665+
qd.window_offset(),
666+
qd.comparison_log_type('ufo_tests'),
650667
]
651668
)
652669

@@ -656,7 +673,8 @@ class TaskQuestions(QuestionContainer, Enum):
656673
list_name="RunJediVariationalExecutable",
657674
questions=[
658675
run_jedi_executable,
659-
qd.perhost()
676+
qd.perhost(),
677+
qd.comparison_log_type('variational'),
660678
]
661679
)
662680

0 commit comments

Comments
 (0)