Bugfix: cases with different time intervals

[sajjadazimi]

Purpose

fix bug

I have:

• followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
• followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
• followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
• checked that this PR does not duplicate an open PR.

In the Content, I have included

• relevant unit tests, and integration tests,
• appropriate docstrings on all functions, structs, and modules, and included relevant documentation.
• I have read and checked the items on the review checklist.

[crocicc]

Looks perfect to me Sajjad! Thank you so much!!

[crocicc]

Here, I think we need to add

        else
            config["model"]["filter"] = make_filter_props(
                config["model"]["filter"]["nz_unfiltered"],
                config["model"]["t_calib"];
                apply = config["model"]["filter"]["apply"],
                nz_per_filtered_cell = config["model"]["filter"]["nz_per_filtered_cell"],
                nt_per_filtered_cell = config["model"]["filter"]["nt_per_filtered_cell"],
            )  
        end

If we check with an example in which the 1st case has the t_cal flag, while the second case doesn't have the t_cal flag, then for the case without the flag it keeps the last filter (the one created for the case with t_cal), not the original one.

Let me know if you agree and if it makes sense to you!

[crocicc]

Doesn't it make a new filter for every case in this way (not considering the default one anymore)?

[sajjadazimi]

I agree and it makes total sense. Thank you!

[crocicc]

I can check with different combinations of cases with and without t_cal, but apart from that I think it looks good!