michaelpb / michaelpb/omnic

Asyncio Improvements

Open
#9 0 comments 0 reactions 0 assignees View on GitHub
enhancement Performance
Dominant language
Python
Stars
16
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Possibly should do this w.r.t. new SubprocessManager

```
converter.py
21 def convert_sync(self, in_resource, out_resource):
22 # TODO: Make both optional (run_until_complete here)
23 msg = 'Converter subclass must override at least convert_sync.'
24 raise NotImplementedError(msg)
```

- [ ] AsyncIO improvements
- [ ] Replace all file system calls with aiofiles
- [ ] Replace all subprocess.run system calls with asyncio equivalent

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with converter.py around convert_sync and review the referenced new SubprocessManager. Locate the filesystem calls and subprocess.run calls covered by the checklist, then determine the async replacements needed. Done means the listed filesystem and subprocess operations use asyncio-compatible approaches and the checklist is complete.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.