illinois-cs241 / broadway-on-demand

Webapp that facilitates on-demand assignment autograding via Broadway.
Other
11 stars 4 forks source link

Add scheduled grading run details table #138

Closed Xiangmingchen closed 3 years ago

Xiangmingchen commented 3 years ago

https://user-images.githubusercontent.com/31719253/112406020-654ce300-8ce1-11eb-821c-d1b4a20fee8f.mp4

Features added

Broadway endpoint integration

Used grading run env endpoint to pair up student netids and grading job ids (thanks to @st-arry for creating the endpoint 🎉). Because this endpoint is guarded by admin tokens, on-demand server acts as a middle man to relay the request results.

Xiangmingchen commented 3 years ago

Note that the getJobStatus function used by "status" button (in details table) gets the grading job status of all jobs within the run. This is because broadway doesn't have an endpoint for querying job status for a specific job. This might cause some performance issues given that a run might have hundreds of jobs in it.

However we shouldn't try to cache the result and reuse it every time because the grading job status might change between clicks. The best way to improve this would require a new endpoint on broadway.

Xiangmingchen commented 3 years ago

Address #69