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.

Sign up for GitHub

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

Jump to bottom

Add Var.split_by_season #79

Merged
merged 6 commits into from
Sep 10, 2024
Merged

Add Var.split_by_season #79

merged 6 commits into from
Sep 10, 2024

Commits on Sep 10, 2024

  1. Fix formatting issue

    @ph-kev
    ph-kev committed Sep 10, 2024
    Configuration menu
    Copy the full SHA
    cdd7a57 View commit details
    Browse the repository at this point in the history

  2. Add Utils._data_at_dim_vals

    @ph-kev
    ph-kev committed Sep 10, 2024
    Configuration menu
    Copy the full SHA
    b406f63 View commit details
    Browse the repository at this point in the history

  3. Add Base.is_empty(var::OutputVar)

    @ph-kev
    ph-kev committed Sep 10, 2024
    Configuration menu
    Copy the full SHA
    7e62598 View commit details
    Browse the repository at this point in the history

  4. Add Utils.time_to_date and Utils.date_to_time 

    Also, add Utils.period_to_seconds_float. These functions are
functionally identical to the equivalent functions in ClimaUtilities. We
do not export from ClimaUtilities because we do not want to add
ClimaUtilities as a dependency.
    @ph-kev
    ph-kev committed Sep 10, 2024
    Configuration menu
    Copy the full SHA
    25f6f33 View commit details
    Browse the repository at this point in the history

  5. Remove Utils.split_by_season from public API 

    We do this because we want to show the documentation for
Var.split_by_season and not Utils.split_by_season.
    @ph-kev
    ph-kev committed Sep 10, 2024
    Configuration menu
    Copy the full SHA
    dd0835f View commit details
    Browse the repository at this point in the history

  6. Add Var.split_by_season 

    Bug introduced: The documentation show the wrong module name for
Var.split_by_season. It is ClimaAnalysis.Utils.split_by_season instead
of ClimaAnalysis.Var.split_by_season.
    @ph-kev
    ph-kev committed Sep 10, 2024
    Configuration menu
    Copy the full SHA
    c9c0eec View commit details
    Browse the repository at this point in the history