Skip to content

Prepare common data model 0.4#13

Merged
Alexander-Barth merged 3 commits into
JuliaGeo:mainfrom
lupemba:Prepare-CommonDataModel-0.4
Oct 16, 2025
Merged

Prepare common data model 0.4#13
Alexander-Barth merged 3 commits into
JuliaGeo:mainfrom
lupemba:Prepare-CommonDataModel-0.4

Conversation

@lupemba

@lupemba lupemba commented Jun 25, 2025

Copy link
Copy Markdown
Contributor

This PR requires: JuliaGeo/CommonDataModel.jl#35

The PR is related to the following issues JuliaGeo/CommonDataModel.jl#32

Content:

  • Bump CommonDataModel to 0.4 (Not merged yet).
  • Replace getindex and setindex with DiskArray readblock! and writeblock!

The PR have been tested offline by

@Alexander-Barth

Copy link
Copy Markdown
Member

@lupemba I just resolved some merge conflicts. If it is ok for you, I can merge the PR.

@lupemba lupemba marked this pull request as ready for review September 16, 2025 15:25
@lupemba

lupemba commented Sep 16, 2025

Copy link
Copy Markdown
Contributor Author

@Alexander-Barth,
It is okay for me to merge the PR but maybe we should just rerun the CI before merging.

@rafaqz rafaqz closed this Oct 16, 2025
@rafaqz rafaqz reopened this Oct 16, 2025
Comment thread src/variable.jl
@rafaqz

rafaqz commented Oct 16, 2025

Copy link
Copy Markdown
Member

@Alexander-Barth is it ok to merge this and bump a new vesion?

@Alexander-Barth Alexander-Barth merged commit 2a6bc08 into JuliaGeo:main Oct 16, 2025
9 checks passed
@Alexander-Barth

Copy link
Copy Markdown
Member

yes, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants