leap-stc / leap-stc/ClimSim

Refactor `climsim_utils.data_utils` into `preprocessing` and `evaluation`

Open
#17 1 comment 0 reactions 1 assignee View on GitHub

@cisaacstern is already working on this.

Since Aug 18, 2023.

enhancement
Dominant language
Jupyter Notebook
Stars
164
Forks
54
PR merge metrics
No merged PRs in 30d

Description

Per @jerrylin96's explanation, data_utils currently performs two functions:

  • preprocessing
  • evaluation

which are largely separable. For maintainability/readability it probably makes sense to refactor these into separate python modules, so the imports would then look like:

from climsim_utils.preprocessing import something
from climsim_utils.evaluation import something_else

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.