-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat(common-lib): derive dynamic field instead of call to fullnode #246
feat(common-lib): derive dynamic field instead of call to fullnode #246
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
CodSpeed Performance ReportMerging #246 will not alter performanceComparing Summary
Benchmarks breakdown
|
965321f
to
2af5519
Compare
This version includes the deriveDynamicFieldID function needed to calculate the DF object ID without making a request to a fullnode.
2af5519
to
c7ee62d
Compare
test name: "should return NOT_FOUND if dynamic fields are not found"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks @Tzal3x . Generally looks good, only one more important comment.
) - Upgrade @mysten/sui to 1.12.0 This version includes the `deriveDynamicFieldID` function needed to calculate the DF object ID without making a request to a fullnode. - Replace `client.getDynamicFieldObject` with `deriveDynamicFieldID`. - Skip `page_fetching` benchmarks. The results we get from it are misleading and slow down development whenever there are changes happening in its' dependencies. In the near future we should revisit how to properly bench this. #250 TODO (following PR) - ⚡ Performance improvement: Use `multiGetObjects` to fetch both the display and the dynamic field object in the same request.
This version includes the
deriveDynamicFieldID
function needed to calculate the DF object ID
without making a request to a fullnode.
client.getDynamicFieldObject
withderiveDynamicFieldID
.page_fetching
benchmarks. The results we get from it are misleading and slow down development whenever there are changes happening in its' dependencies. In the near future we should revisit how to properly bench this. Use multiGetObjects to fetch both the display and the dynamic field object in the same request #250TODO (following PR)
multiGetObjects
to fetch both the display and the dynamic field object in the same request.