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

simplify back-end selection in brun, and fix auto-detection #92

Merged
merged 1 commit into from
Oct 20, 2022

Conversation

arvidn
Copy link
Contributor

@arvidn arvidn commented Oct 20, 2022

If you specify a back-end explicitly, it's better to fail than to silently fall back to the other back-end.
Specifically, there is a problem ensuring stage is 0. This also fixes that.

@arvidn arvidn requested a review from richardkiss October 20, 2022 15:17
@arvidn arvidn force-pushed the simplify-brun-backend branch from 06f4b6e to dba35ff Compare October 20, 2022 15:22
clvm_tools/cmds.py Outdated Show resolved Hide resolved
ChiaMineJP
ChiaMineJP previously approved these changes Oct 20, 2022
clvm_tools/cmds.py Outdated Show resolved Hide resolved
clvm_tools/cmds.py Outdated Show resolved Hide resolved
@arvidn arvidn force-pushed the simplify-brun-backend branch from 6525a9a to 4b43797 Compare October 20, 2022 19:52
@arvidn arvidn merged commit c796b45 into main Oct 20, 2022
@arvidn arvidn deleted the simplify-brun-backend branch October 20, 2022 20:40
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.

4 participants