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

Switch to using IntMap for TxSeq decoding #4900

Open
lehins opened this issue Feb 20, 2025 · 0 comments · May be fixed by #4904
Open

Switch to using IntMap for TxSeq decoding #4900

lehins opened this issue Feb 20, 2025 · 0 comments · May be fixed by #4904
Assignees
@teodanciu
Labels
💳 technical-debt Issues related to technical debt we introduced

Comments

@lehins
Copy link
Collaborator

lehins commented Feb 20, 2025

IntMap is a bit more efficient than Map when dealing with Int key.
It will be better to use IntMap for decoding TxAuxData and in usage of indexLookupSeq
@lehins lehins added the 💳 technical-debt Issues related to technical debt we introduced label Feb 20, 2025
@lehins lehins mentioned this issue Feb 20, 2025
Merged
9 tasks
@teodanciu teodanciu linked a pull request Feb 21, 2025 that will close this issue
Open
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@teodanciu teodanciu

Labels
💳 technical-debt Issues related to technical debt we introduced
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve TxSeq decoders IntersectMBO/cardano-ledger
2 participants
@teodanciu @lehins