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

Integrate new version of Atree with register inlining and data deduplication #2809

Closed
fxamacker opened this issue Sep 22, 2023 · 1 comment
Closed

Integrate new version of Atree with register inlining and data deduplication #2809

fxamacker opened this issue Sep 22, 2023 · 1 comment
Assignees
@turbolent @fxamacker
Labels
Feature Feedback

Comments

@fxamacker
Copy link
Member

Issue to be solved

Atree register inlining and encoding size optimizations introduced in onflow/atree#342, etc. required breaking changes to Atree's API, so existing Cadence code needs to be updated to use it.

Suggested Solution

TODO:

Adding tests for this requires expertise in Cadence internals.
This was referenced Oct 5, 2023
Merged
Merged
@fxamacker fxamacker changed the title Integrate new version of Atree with register inlining Integrate new version of Atree with register inlining and data deduplication Oct 18, 2023
This was referenced Oct 18, 2023
Merged
Merged
@j1010001 j1010001 mentioned this issue Nov 16, 2023
Closed
@fxamacker
Copy link
Member Author

Closed by #2882 and other PRs.

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

@turbolent turbolent

@fxamacker fxamacker

Labels
Feature Feedback
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@turbolent @fxamacker