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.

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

[DO NOT MERGE] Bump atree version to use updated branch of fxamacker/cbor #3818

Open
wants to merge 1 commit into
base: v1.3
Choose a base branch
from

Conversation

fxamacker
Copy link
Member

@fxamacker fxamacker commented Mar 21, 2025

Currently, atree is using a feature branch of fxamacker/cbor that was based on cbor v2.4.0.

Anchorage Digital has dependencies requiring fxamacker/cbor v2.7.0, so they were encountering build error because atree and cadence were using fxamacker/cbor branch based on v2.4.0.

This PR bumps versions to use the updated feature/stream-mode branch of fxamacker/cbor that is backward compatible with v2.7.0 while also having extra CBOR streaming features we use.


  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@fxamacker fxamacker self-assigned this Mar 21, 2025
Currently, atree is using a feature branch of fxamacker/cbor
that was based on cbor v2.4.0.

This commit bumps version of fxamacker/cbor to use the
updated feature/stream-mode branch based on fxamacker/cbor v2.7.0.
@fxamacker fxamacker force-pushed the fxamacker/bump-atree-version-to-use-updated-cbor-branch branch from 749d9e7 to 1f1659d Compare March 21, 2025 22:28
Copy link

Cadence Benchstat comparison

This branch with compared with the base branch onflow:v1.3 commit 6d0a2c9
The command for i in {1..N}; do go test ./... -run=XXX -bench=. -benchmem -shuffle=on; done was used.
Bench tests were run a total of 7 times on each branch.

Collapsed results for better readability

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant