CliMA / CliMA/ParameterEstimocean.jl

Unintentional warning after each first-time step of any simulation?

Open
#219 5 comments 0 reactions 0 assignees View on GitHub
❓ question 🚰 user experience
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.