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

Improve clarity-wasm behaviour in console #1306

Closed
3 tasks
hugocaillard opened this issue Jan 12, 2024 · 1 comment · Fixed by #1330
Closed
3 tasks

Improve clarity-wasm behaviour in console #1306

hugocaillard opened this issue Jan 12, 2024 · 1 comment · Fixed by #1330
Assignees

Comments

@hugocaillard
Copy link
Collaborator

clarity-wasm can show some stability issues so it should be possible to opt-out of clarity-wasm but keep it enabled by default to really tests clarity-wasm.

Also, it could be silent by default (avoid logging the timings).

The error message when clarity-wasm returns a different result than the interpreter can be improved as well, explain that there's is flag to disable clarity-wasm and asking to open an issue with the log output.

  • add a --no-clarity-wasm flag to clarinet console
  • add a --show-clarity-wasm-perf flag to clarinet console
  • improve clarity-wasm warning message when it returns a different result
@hugocaillard
Copy link
Collaborator Author

Note that all of the clarity-wasm transactions are made from an other address than the default deployer.
It can lead to some weird behavior (like the funds of each addresses can differ).
Make sure it's not to confusing or error prone

See #1266 (comment)

@hugocaillard hugocaillard moved this from 📋 Backlog to 🏗 In Progress in DevTools Jan 22, 2024
@hugocaillard hugocaillard self-assigned this Jan 22, 2024
@hugocaillard hugocaillard linked a pull request Jan 30, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this from 🏗 In Progress to ✅ Done in DevTools Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant