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

Introduce blocking circuit create & prove #2

Merged
merged 3 commits into from
Jan 14, 2025

Conversation

katiemckeon
Copy link
Collaborator

🔄 Pull Request

📝 Description

For some use cases, an async circuit create or prove method doesn't fit within a user's codebase. Here, we introduce simple blocking analogues of circuit_create and circuit_prove as convenience methods. They spawn a runtime so its not a perfect fit for all applications but it balances to the tradeoff of adding a convenience method without a bunch of complexity. GEFN.

🔗 Related Tickets & Documents

📋 Type of PR (check all applicable)

  • 🔨 Refactor
  • ✨ Feature
  • 🐛 Bug Fix
  • ⚡ Optimization
  • 📚 Documentation Update

💭 Developer Notes

Important note: during testing of this PR, I uncovered some issues with docs testing and suppressed them. That would be any of the ignore additions. Created an issue to come back to when we are at the stage of publishing docs.

🧪 Testing Instructions

Integration tests include both new blocking methods and pass.

✅ Testing Status

  • Yes, tests added/updated
  • No tests needed because: please explain why
  • Help needed with testing

@katiemckeon katiemckeon merged commit ea66645 into main Jan 14, 2025
1 check passed
@katiemckeon katiemckeon deleted the klm-introduce-blocking-methods branch January 14, 2025 02:51
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.

1 participant