-
Notifications
You must be signed in to change notification settings - Fork 190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug in interpolate!(target_fts::FieldTimeSeries, source_fts::FieldTimeSeries)
?
#3731
Comments
Shouldn't that be |
Can you run the function (produce a bug with code rather than merely by reading it)? Is there a test? |
The tests in ClimaOcean run the function and produced a bug. I linked the log above. I can try to create an MWE if it helps. |
#3723 seems to have deleted |
Wouldn't the MWE be a one line function? We also need to test it here |
#3733 introduces a very simple test for interpolation on field time series (testing trivial interpolation) |
good |
* add the bugfix * bump patch release * add interpolate * fix tests --------- Co-authored-by: Navid C. Constantinou <[email protected]>
Seems that there is a bug here?
Oceananigans.jl/src/OutputReaders/field_time_series_indexing.jl
Line 205 in 5218acd
target_times
is never defined but used as an arg in the kernel?This produces the error in the ClimaOcean CI; see https://buildkite.com/clima/climaocean-ci/builds/1264#01918d21-30e8-4d9c-9ee8-a220eb16eb28/202-514
cc @simone-silvestri
The text was updated successfully, but these errors were encountered: