autolab / Autolab

Course management service that enables auto-graded programming assignments.
http://www.autolabproject.com/
Apache License 2.0
765 stars 220 forks source link

DK displaying gradebook #69

Closed droh closed 9 years ago

droh commented 9 years ago

Seth uploaded the roster and then visited the gradebook, which DK'd attempting to run the courseAverage function. I fixed it by manually reloading the course.rb file, but the fact that I had to do it is a bug.

ymzong commented 9 years ago

I cannot reproduce the bug in Seth's course after loading a user with a dummy roster -- can someone try reproduce it and post the error message?

dlbucci commented 9 years ago

@droh was this fixed when the permissions on the course directory were fixed?

droh commented 9 years ago

No, his permission bits were fine. This was fixed when I reloaded the course.rb file, but it shouldn't have happened. Somehow, we want to ensure that the course.rb file is always loaded when a course is created.

-- David O'Hallaron -- Prof of CS and ECE, Carnegie Mellon University -- GHC 7517, x8-8199 -- http://www.cs.cmu.edu/~droh

On Sun, Jan 18, 2015 at 3:59 PM, Daniel Bucci notifications@github.com wrote:

@droh https://github.com/droh was this fixed when the permissions on the course directory were fixed?

— Reply to this email directly or view it on GitHub https://github.com/autolab/Autolab/issues/69#issuecomment-70425652.

andrewwuan commented 9 years ago

@droh Just to make sure, the reload should be done after "a course a created" rather than someone "upload a roster" right?

droh commented 9 years ago

Yes, course.rb reload should be done after course created.

On Monday, January 19, 2015, An Wu notifications@github.com wrote:

@droh https://github.com/droh Just to make sure, the reload should be done after "a course a created" rather than someone "upload a roster" right?

— Reply to this email directly or view it on GitHub https://github.com/autolab/Autolab/issues/69#issuecomment-70574660.

Sent from my iPhone.