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

Revisit BatchManager #536

Open
whyitfor opened this issue Dec 14, 2024 · 0 comments
Open

Revisit BatchManager #536

whyitfor opened this issue Dec 14, 2024 · 0 comments
Assignees

Comments

@whyitfor
Copy link
Contributor

The BatchManager seems to duplicate functionality provide by asyncio.Semaphore.

Goal of this issue is to investigate whether it can be replaced entirely by a Semaphore. The goal of this is to simplify ofrak_io and make the OFRAK code base incrementally more maintainable.

Which files would be affected?
The package ofrak_io, ofrak_ghidra`.

Does the proposed maintenance include non-doc string functional changes to the Python code?
Potentially -- it would remove the BatchManager interface and update the ofrak_ghidra implementation.

Are you interested in implementing it yourself?
No

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

No branches or pull requests

2 participants