NatLabRockies / NatLabRockies/chronify

Support variable time array lengths

Open
#43 2 comments 0 reactions 1 assignee View on GitHub

@lixiliu is already working on this.

Since Feb 13, 2025.

Dominant language
Python
Stars
5
Forks
8
PR merge metrics
No merged PRs in 30d

Description

Chronify currently requires that all time arrays have the same length. This is fine for datasets based on a single [weather] year. It doesn't work across years because of leap years. The problem exists for datetime and index-time configurations.

Suppose a dataset like this:

  • Years 2010 - 2020
  • Index time, hourly
  • 2012, 2016, 2020 will have 8784 indexes
  • Other years will have 8760
  1. What does the user specify for length in the time config?
  2. How does chronify generate correct expected time index values?

The time configs need additional parameters to specify these behaviors. We can assume that the dataset will have a column that can be used to distinguish years (like "year" or "weather_year").

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.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.