Skip to content

Fix concurrency bug in spotrna#49

Merged
tkaragianes merged 1 commit intomasterfrom
fix/spotrna-concurrency
Jan 25, 2025
Merged

Fix concurrency bug in spotrna#49
tkaragianes merged 1 commit intomasterfrom
fix/spotrna-concurrency

Conversation

@luxaritas
Copy link
Contributor

If multiple processes called spotrna at the same time, spotrna internally would internally (over)write a file causing one or all of the predictions to use/return the wrong data

If multiple processes called spotrna at the same time, spotrna internally would internally (over)write a file causing one or all of the predictions to use/return the wrong data
@tkaragianes tkaragianes merged commit 9a65ddd into master Jan 25, 2025
3 checks passed
@luxaritas
Copy link
Contributor Author

For future reference, the behavior in the spotrna codebase that this triggered can be found here:
https://github.com/jaswindersingh2/SPOT-RNA/blob/master/SPOT-RNA.py#L28C1-L28C11
https://github.com/jaswindersingh2/SPOT-RNA/blob/master/SPOT-RNA.py#L46

@luxaritas luxaritas deleted the fix/spotrna-concurrency branch February 24, 2025 23:19
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.

2 participants