Skip to content

Commit

Permalink
Don't test polygonize with OffsetArrays
Browse files Browse the repository at this point in the history
  • Loading branch information
asinghvi17 authored Jul 29, 2024
1 parent ca428c6 commit 5e6ccd9
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions test/methods/polygonize.jl
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
using GeometryOps, GeoInterface, Test
import OffsetArrays, DimensionalData, Rasters
import DimensionalData, Rasters
# import OffsetArrays

# Missing holes throw a warning, so testing there are
# no warnings in a range of randomisation is one way to test
Expand Down Expand Up @@ -41,6 +42,11 @@ end


@testset "Polygonize with exotic arrays" begin
# OffsetArrays does a bit of undesirable piracy,
# so we test without it for now.
# See https://github.com/JuliaGeo/GeometryOps.jl/issues/187
# and https://github.com/JuliaArrays/OffsetArrays.jl/issues/306
#=
@testset "OffsetArrays" begin
data = rand(1:4, 100, 100) .== 1
evil = OffsetArrays.Origin(-100, -100)(data)
Expand All @@ -51,6 +57,7 @@ end
end
@test GO.equals(data_mp, evil_in_data_space_mp)
end
=#
@testset "DimensionalData" begin
data = rand(1:4, 100, 100) .== 1
evil = DimensionalData.DimArray(data, (DimensionalData.X(1:100), DimensionalData.Y(1:100)))
Expand All @@ -66,4 +73,4 @@ end
@test GO.equals(data_mp, evil_mp)
@test GI.crs(evil_mp) == GI.crs(evil)
end
end
end

0 comments on commit 5e6ccd9

Please sign in to comment.