-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add ability to automatically retry failed multiple questions #608
Conversation
BREAKING CHANGE: Use `datafile.local_path` instead of `datafile.path`; use `octue.utils.processes.run_logged_subprocess` instead of `octue.utils.threads.run_logged_subprocess`
@cortadocodes I have been using this branch in power-loss-service and it is working very well. Perhaps we could think of some additional options to make it a bit more useful. Like:
We used to use a library called dramatiq for working with workers and async tasks and it had similar options. Dramatiq doc for reference here |
That's great to hear @nvn-nil! Thanks for your feedback, I'll incorporate it and release soonish |
Contents (#608)
IMPORTANT: There is 1 breaking change.
Enhancements
Child.ask_multiple
python3.11
baseOperations
Refactoring
Testing
TestChild
testsUpgrade instructions
💥 Remove deprecated code
datafile.local_path
instead ofdatafile.path
octue.utils.processes.run_logged_subprocess
instead ofoctue.utils.threads.run_logged_subprocess