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

Remove ScriptCostLimit #757

Closed
kushti opened this issue Dec 13, 2021 · 0 comments · Fixed by #896
Closed

Remove ScriptCostLimit #757

kushti opened this issue Dec 13, 2021 · 0 comments · Fixed by #896
Milestone
v5.x

Comments

@kushti
Copy link
Member

kushti commented Dec 13, 2021

ScriptCostLimit in SigmaConstants used in tests and profiler only, so not really a hard-coded constant. Replace it with some other value, e.g. val costLimitForTests somewhere
@aslesarenko aslesarenko mentioned this issue Dec 17, 2021
Merged
21 tasks
@aslesarenko aslesarenko added this to the v5.x milestone Dec 22, 2021
@aslesarenko aslesarenko mentioned this issue Aug 8, 2023
Merged
6 tasks
@aslesarenko aslesarenko linked a pull request Aug 9, 2023 that will close this issue
Refactoring and improvments #896
Merged
6 tasks
@aslesarenko aslesarenko mentioned this issue Aug 31, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v5.x
Development

Successfully merging a pull request may close this issue.

Refactoring and improvments ergoplatform/sigmastate-interpreter
2 participants
@aslesarenko @kushti