LoadError: ArgumentError: Chunk sizes must be strictly positive

[ctroupin]

• Got this error during the automated tests after a git push.
• Not sure how to reproduce.
• Mentionned for documentation purpose.

...
reconstruct gridded data: Error During Test at /home/runner/work/DINCAE.jl/DINCAE.jl/test/runtests.jl:6
  Got exception outside of a @test
  LoadError: ArgumentError: Chunk sizes must be strictly positive
  Stacktrace:
    [1] RegularChunks
      @ ~/.julia/packages/DiskArrays/QlfRF/src/chunks.jl:26 [inlined]
    [2] #24
      @ ~/.julia/packages/DiskArrays/QlfRF/src/chunks.jl:133 [inlined]
    [3] map (repeats 3 times)
      @ ./tuple.jl:318 [inlined]
    [4] DiskArrays.GridChunks(a::Tuple{Int64, Int64, Int64}, chunksize::Tuple{Int64, Int64, Int64}; offset::Tuple{Int64, Int64, Int64})
      @ DiskArrays ~/.julia/packages/DiskArrays/QlfRF/src/chunks.jl:132
    [5] GridChunks
      @ ~/.julia/packages/DiskArrays/QlfRF/src/chunks.jl:131 [inlined]
    [6] #GridChunks#18
      @ ~/.julia/packages/DiskArrays/QlfRF/src/chunks.jl:130 [inlined]
    [7] DiskArrays.GridChunks(a::NCDatasets.Variable{Float32, 3, NCDataset{Nothing}}, chunksize::Tuple{Int64, Int64, Int64})
      @ DiskArrays ~/.julia/packages/DiskArrays/QlfRF/src/chunks.jl:130
    [8] eachchunk(v::NCDatasets.Variable{Float32, 3, NCDataset{Nothing}})
      @ NCDatasets ~/.julia/packages/NCDatasets/vRl1m/src/variable.jl:407
    [9] _writeblock!(::NCDatasets.Variable{Float32, 3, NCDataset{Nothing}}, ::Array{Float32, 3}, ::Base.OneTo{Int64}, ::Vararg{AbstractVector})
      @ DiskArrays ~/.julia/packages/DiskArrays/QlfRF/src/batchgetindex.jl:187
   [10] writeblock!(::NCDatasets.Variable{Float32, 3, NCDataset{Nothing}}, ::Array{Float32, 3}, ::Base.OneTo{Int64}, ::Vararg{AbstractVector})
      @ NCDatasets ~/.julia/packages/DiskArrays/QlfRF/src/batchgetindex.jl:213
   [11] setindex_disk!(::NCDatasets.Variable{Float32, 3, NCDataset{Nothing}}, ::Matrix{Float32}, ::Function, ::Vararg{Any})
      @ DiskArrays ~/.julia/packages/DiskArrays/QlfRF/src/diskarray.jl:57
   [12] setindex!(::NCDatasets.Variable{Float32, 3, NCDataset{Nothing}}, ::Matrix{Float32}, ::Function, ::Function, ::Int64)
      @ NCDatasets ~/.julia/packages/DiskArrays/QlfRF/src/diskarray.jl:229
   [13] savesample(ds::NCDataset{Nothing}, varnames::Vector{String}, xrec::Array{Float64, 4}, meandata::Array{Float32, 3}, ii::Int64, offset::Int64; output_ndims::Int64)
      @ DINCAE ~/work/DINCAE.jl/DINCAE.jl/src/data.jl:573
   [14] macro expansion
      @ ~/work/DINCAE.jl/DINCAE.jl/src/model.jl:536 [inlined]
   [15] macro expansion
      @ ./timing.jl:273 [inlined]
   [16] macro expansion
      @ ~/work/DINCAE.jl/DINCAE.jl/src/model.jl:526 [inlined]
   [17] macro expansion
      @ ./timing.jl:273 [inlined]
   [18] reconstruct(Atype::Type, data_all::Vector{Vector{NamedTuple{(:filename, :varname, :obs_err_std, :jitter_std, :isoutput), Tuple{String, String, Int64, Float64, Bool}}}}, fnames_rec::Vector{String}; epochs::Int64, batch_size::Int64, truth_uncertain::Bool, enc_nfilter_internal::Vector{Int64}, skipconnections::UnitRange{Int64}, clip_grad::Float64, regularization_L1_beta::Int64, regularization_L2_beta::Int64, save_epochs::Vector{Int64}, is3D::Bool, upsampling_method::Symbol, ntime_win::Int64, learning_rate::Float64, learning_rate_decay_epoch::Float64, min_std_err::Float64, loss_weights_refine::Tuple{Float64}, cycle_periods::Tuple{Float64}, output_ndims::Int64, direction_obs::Nothing, remove_mean::Bool, paramfile::String, laplacian_penalty::Int64, laplacian_error_penalty::Int64)
      @ DINCAE ~/work/DINCAE.jl/DINCAE.jl/src/model.jl:511
   [19] top-level scope
      @ ~/work/DINCAE.jl/DINCAE.jl/test/test_DINCAE_SST.jl:63
   [20] include(fname::String)
      @ Base.MainInclude ./client.jl:478
   [21] macro expansion
      @ ~/work/DINCAE.jl/DINCAE.jl/test/runtests.jl:7 [inlined]
   [22] macro expansion
      @ /opt/hostedtoolcache/julia/1.9.4/x64/share/julia/stdlib/v1.9/Test/src/Test.jl:1498 [inlined]
   [23] top-level scope
      @ ~/work/DINCAE.jl/DINCAE.jl/test/runtests.jl:7
   [24] include(fname::String)
      @ Base.MainInclude ./client.jl:478
   [25] top-level scope
      @ none:6
   [26] eval
      @ ./boot.jl:370 [inlined]
   [27] exec_options(opts::Base.JLOptions)
      @ Base ./client.jl:280
   [28] _start()
      @ Base ./client.jl:522
  in expression starting at /home/runner/work/DINCAE.jl/DINCAE.jl/test/test_DINCAE_SST.jl:52
Test Summary:            | Error  Total     Time
reconstruct gridded data |     1      1  1m47.0s
ERROR: LoadError: Some tests did not pass: 0 passed, 0 failed, 1 errored, 0 broken.
in expression starting at /home/runner/work/DINCAE.jl/DINCAE.jl/test/runtests.jl:6
ERROR: LoadError: Package DINCAE errored during testing
Stacktrace:
 [1] pkgerror(msg::String)
   @ Pkg.Types /opt/hostedtoolcache/julia/1.9.4/x64/share/julia/stdlib/v1.9/Pkg/src/Types.jl:69
 [2] test(ctx::Pkg.Types.Context, pkgs::Vector{Pkg.Types.PackageSpec}; coverage::Bool, julia_args::Cmd, test_args::Cmd, test_fn::Nothing, force_latest_compatible_version::Bool, allow_earlier_backwards_compatible_versions::Bool, allow_reresolve::Bool)
   @ Pkg.Operations /opt/hostedtoolcache/julia/1.9.4/x64/share/julia/stdlib/v1.9/Pkg/src/Operations.jl:2019
 [3] test
   @ /opt/hostedtoolcache/julia/1.9.4/x64/share/julia/stdlib/v1.9/Pkg/src/Operations.jl:1900 [inlined]
 [4] test(ctx::Pkg.Types.Context, pkgs::Vector{Pkg.Types.PackageSpec}; coverage::Bool, test_fn::Nothing, julia_args::Vector{String}, test_args::Cmd, force_latest_compatible_version::Bool, allow_earlier_backwards_compatible_versions::Bool, allow_reresolve::Bool, kwargs::Base.Pairs{Symbol, IOContext{Base.PipeEndpoint}, Tuple{Symbol}, NamedTuple{(:io,), Tuple{IOContext{Base.PipeEndpoint}}}})
   @ Pkg.API /opt/hostedtoolcache/julia/1.9.4/x64/share/julia/stdlib/v1.9/Pkg/src/API.jl:441
 [5] test(pkgs::Vector{Pkg.Types.PackageSpec}; io::IOContext{Base.PipeEndpoint}, kwargs::Base.Pairs{Symbol, Any, Tuple{Symbol, Symbol, Symbol}, NamedTuple{(:coverage, :julia_args, :force_latest_compatible_version), Tuple{Bool, Vector{String}, Bool}}})
   @ Pkg.API /opt/hostedtoolcache/julia/1.9.4/x64/share/julia/stdlib/v1.9/Pkg/src/API.jl:156
 [6] test(; name::Nothing, uuid::Nothing, version::Nothing, url::Nothing, rev::Nothing, path::Nothing, mode::Pkg.Types.PackageMode, subdir::Nothing, kwargs::Base.Pairs{Symbol, Any, Tuple{Symbol, Symbol, Symbol}, NamedTuple{(:coverage, :julia_args, :force_latest_compatible_version), Tuple{Bool, Vector{String}, Bool}}})
   @ Pkg.API /opt/hostedtoolcache/julia/1.9.4/x64/share/julia/stdlib/v1.9/Pkg/src/API.jl:171
 [7] top-level scope
   @ ~/work/_actions/julia-actions/julia-runtest/v1/test_harness.jl:15
 [8] include(fname::String)
   @ Base.MainInclude ./client.jl:478
 [9] top-level scope
   @ none:1
in expression starting at /home/runner/work/_actions/julia-actions/julia-runtest/v1/test_harness.jl:7
Error: Process completed with exit code 1.
``

[Alexander-Barth]

the upcoming NCDataset 0.13.2 should fix this, (at least in my local test :-))

JuliaRegistries/General#97148