Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • O Open ZGY
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 4
    • Issues 4
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Open Subsurface Data Universe SoftwareOpen Subsurface Data Universe Software
  • Platform
  • Domain Data Management Services
  • Seismic
  • Open ZGY
  • Merge requests
  • !66

Allow appending to an empty file created by the old accessor.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Paal Kvamme requested to merge kvamme62/old-accessor-alignment into master May 27, 2021
  • Overview 0
  • Commits 2
  • Pipelines 4
  • Changes 8

For on-prem files, OpenZGY should be able to open the file for update regardless of whether it was written by the old or the new accessor. This didn't quite work due to differences in padding.

The solution is to add padding before the start of every uncompressed brick to align it with the brick size. This happens in addition to the current code to add padding at the end of the header area.

In most cases this will not make any difference. After the header has been padded, and if only uncompressed bricks are written, they will all end up aligned anyway.

If updating an empty file created by the old ZGY accessor the change is needed to avoid ending up with an uncompressed version 3 ZGY file with unaligned data bricks. In addition to violating the rule that v3 bricks are aligned, this case also caused an error when the unpadded header was updated by OpenZGY with a padded header. Thereby overwriting part of the first data brick.

If a file is written with a mix of compressed and uncompressed data bricks this change might waste some space on the file. This is unlikely to become a real problem.

Added a unit test to verify the fix.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: kvamme62/old-accessor-alignment