Skip to content
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.

Sign up for GitHub

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

Jump to bottom

autotest/gdrivers/rmf.py::test_rmf_31e randomly fails #4399

Closed
rouault opened this issue Aug 31, 2021 · 3 comments
Closed

autotest/gdrivers/rmf.py::test_rmf_31e randomly fails #4399

rouault opened this issue Aug 31, 2021 · 3 comments

Comments

@rouault
Copy link
Member

rouault commented Aug 31, 2021

e.g in https://github.com/OSGeo/gdal/pull/4398/checks?check_run_id=3473089379
The random nature of the failure is likely due to this tests using random numbers. I strongly suspect this is due to the delta encoding of the values with a variable integer strategy in the RMFDem compressor, and for some deltas inappropriately being classified. Or using one of the reserved value.
@rouault
Copy link
Member Author

rouault commented Aug 31, 2021

CC @drons

@drons
Copy link
Contributor

drons commented Aug 31, 2021

CC @drons

Ok, I'll check it out.

drons added a commit to drons/gdal that referenced this issue Sep 1, 2021
@drons
RMF: Rework RMF delta-compression test (OSGeo#4399)
86d711e
drons added a commit to drons/gdal that referenced this issue Sep 1, 2021
@drons
RMF: Fix delta compression with non-initialized elevation min-max (OS…
ecd9506 
…Geo#4399)
rouault added a commit that referenced this issue Sep 1, 2021
@rouault
Merge pull request #4405 from drons/devRmfDemTestFix
f527fbd 
RMF: Rework RMF delta-compression test (#4399)
github-actions bot pushed a commit that referenced this issue Sep 1, 2021
@rouault @github-actions
Merge pull request #4405 from drons/devRmfDemTestFix
af4d35b 
RMF: Rework RMF delta-compression test (#4399)
rouault added a commit that referenced this issue Sep 7, 2021
@rouault
Merge pull request #4413 from OSGeo/backport-4405-to-release/3.3
fda7f47 
[Backport release/3.3] RMF: Rework RMF delta-compression test (#4399)
@rouault
Copy link
Member Author

rouault commented Nov 22, 2021

Seems to be fixed

@rouault rouault closed this as completed Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rouault @drons