-
Notifications
You must be signed in to change notification settings - Fork 3
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
Multiprocess Grading #21
Comments
I don't see why not. If I remember, grader uses a bunch of async junk. It shouldn't be hard to do. The Python part wouldn't be parallel -- just concurrent. The student's assignments would be running in subprocesses anyway, so those should run in parallel. |
Output here would be a challenge... Logging output is currently done live--- would it make more sense for async to just output all at once and log when it starts/ends? |
Additionally--- isn't this as simple as wrapping up: https://github.com/redkyn/grader/blob/master/grader/grader/commands/grade.py#L42-L43 With a coroutine lambda then waiting w/ asyncio? |
@brhoades I believe it's roughly that simple. It just needs an event loop and some coroutines. Hopefully, |
Yep, those are my thoughts too. Default concurrency amount can be read from a gradesheet but defaults to the number of cores or something. I'll take a look at this due to its apparent simplicity sometime soon. |
Sometimes my lab gets cold and I don't have a space heater under my desk. Multiprocess grading would help all my CPU cores bring about the heat death of the universe faster!
EDIT: this would also make grader run faster
The text was updated successfully, but these errors were encountered: