CliMA / CliMA/ParameterEstimocean.jl
Unintentional warning after each first-time step of any simulation?
- Dominant language
- Julia
- Stars
- 22
- Forks
- 6
- PR merge metrics
- No merged PRs in 30d
Description
This warning appears after the initial time-step of any simulation...
https://github.com/CliMA/OceanTurbulenceParameterEstimation.jl/blob/30d5cfbedad1cab328885bce485d19cb5b6afe03/src/Observations.jl#L310-L316
```julia
[ Info: Initializing simulation...
[ Info: ... simulation initialization complete (830.369 ms)
[ Info: Executing initial time step...
[ Info: ... initial time step complete (8.342 ms).
┌ Warning: Current time 20 seconds not found in
│ time collector times ["0 seconds", "1 hour", "2 hours", "3 hours", "4 hours", "5 hours", "6 hours", "7 hours", "8 hours", "9 hours", "10 hours"]
```
I don't think that was the intention, right?
cc @glwagner
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reproducing the warning and inspect src/Observations.jl at lines 310-316, where the time collector lookup is reported. Confirm the expected handling of the initial time step and verify that running a simulation no longer emits this warning for a valid initial time.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100