CliMA / CliMA/ClimaAnalysis.jl
Add `Var.convert_units!`
Open
Beginner friendly
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