Skip to content

Draft Checksums Change#34

Draft
ndp-opendap wants to merge 9 commits into
mainfrom
checksums_update
Draft

Draft Checksums Change#34
ndp-opendap wants to merge 9 commits into
mainfrom
checksums_update

Conversation

@ndp-opendap

@ndp-opendap ndp-opendap commented Jul 11, 2025

Copy link
Copy Markdown
Contributor
  • For the dap4 data response, defines bit 3 (the fourth bit) of the first chunk header as a flag to indicate the presences of checksums in the serialized data.
  • Small reorganization to the Checksums discussion to capture the new checksums flag.

@DennisHeimbigner DennisHeimbigner left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you also need to change line 1854?

with the encoding 0 = data, 1 = end, 2 = error, 4 = Little-Endian */

Comment thread 01_data-model-and-serialized-rep.adoc Outdated
Comment thread 01_data-model-and-serialized-rep.adoc Outdated
@ndp-opendap ndp-opendap requested a review from Mikejmnez August 27, 2025 22:08
@ndp-opendap

ndp-opendap commented Aug 27, 2025

Copy link
Copy Markdown
Contributor Author

Do you also need to change line 1854?

with the encoding 0 = data, 1 = end, 2 = error, 4 = Little-Endian */

Yes thanks, change made...

Comment thread 01_data-model-and-serialized-rep.adoc Outdated
Comment thread 01_data-model-and-serialized-rep.adoc Outdated
Comment thread 01_data-model-and-serialized-rep.adoc Outdated

@jgallagher59701 jgallagher59701 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a good change. Ready to merge.

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.

4 participants