JuliaClimate / JuliaClimate/ClimateBase.jl

rework of temporal sampling

Open
#91 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

breaking hard temporal
Dominant language
Julia
Stars
38
Forks
3
PR merge metrics
No merged PRs in 30d

Description

I think we can rework the source code that handles temporal sampling, to return units that are multiples of existing Julia types. E.g. Hour(1) for hourly sampling, Day(1) for daily, Hour(3) for 3-hourly sampling, etc. This will not only bring more type stability, but also reduce amount of source code as well allow naturally handling arbitrary multiples of such samplings, such as weekly Day(7), or centurial Year(100).

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the source code that handles temporal sampling and inspect how hourly, daily, and arbitrary-multiple units are currently represented. Compare the existing behavior with Julia's Hour, Day, and Year types. Done means sampling returns appropriately typed units for hourly, daily, multi-hour, weekly, and larger multiples.

Written by the indexing model from the issue text.

Assessment

Tech stack
julia
Domain
data
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.