CliMA / CliMA/ClimaAnalysis.jl

Add `Var.convert_units!`

Open Beginner friendly
#422 0 comments 0 reactions 0 assignees View on GitHub
good first issue
Dominant language
Julia
Stars
12
Forks
5
Avg merge
6d 17h
Merged PRs (30d)
3

Description

The non in-place version already exists as `Var.convert_units`. It would be good to add an in-place version to avoid unnecessary allocations.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating Var.convert_units and tracing how the existing non-in-place conversion handles its arguments and unit changes. Add the in-place Var.convert_units! counterpart so callers can convert without unnecessary allocations, then verify that it mutates the expected Var and preserves the behavior of Var.convert_units.

Written by the indexing model from the issue text.

Assessment

Tech stack
julia
Domain
data
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.