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

Speedup loading network configuration #976

Merged
merged 1 commit into from
Aug 29, 2022
Merged

Speedup loading network configuration #976

merged 1 commit into from
Aug 29, 2022

Conversation

lukaw3d
Copy link
Member

@lukaw3d lukaw3d commented Aug 18, 2022

Before: 78+64+740+75 ms
After: 612 ms

@lukaw3d lukaw3d requested review from buberdds and pro-wh August 18, 2022 02:47
@github-actions
Copy link

github-actions bot commented Aug 18, 2022

MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ EDITORCONFIG editorconfig-checker 3 0 0.01s
✅ GIT git_diff yes no 0.01s
✅ TYPESCRIPT eslint 2 0 0 5.02s

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

const minimumStakingAmount = Number(oasis.quantity.toBigInt(stakingParams.min_delegation)) / 10 ** 9
const { epoch, chainContext } = yield* all({
epoch: call([nic, nic.beaconGetEpoch], oasis.consensus.HEIGHT_LATEST),
chainContext: call([nic, nic.consensusGetChainContext]),
Copy link
Member Author

@lukaw3d lukaw3d Aug 18, 2022

Choose a reason for hiding this comment

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

Not sure why this request is so slow (612 ms) 🤷

@buberdds
Copy link
Contributor

snapshot is failing

@lukaw3d lukaw3d force-pushed the lw/speedup-network branch from 46c86d0 to b19e74b Compare August 29, 2022 19:00
@lukaw3d lukaw3d force-pushed the lw/speedup-network branch from b19e74b to b2fa993 Compare August 29, 2022 19:01
@lukaw3d lukaw3d merged commit b1f5e81 into master Aug 29, 2022
@lukaw3d lukaw3d deleted the lw/speedup-network branch August 29, 2022 19:21
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.

2 participants