lsuits / OBSOLETE--DO-NOT-USE--gradebook_moodle

LSU's fork of Moodle for gradebook changes
GNU General Public License v3.0
2 stars 1 forks source link

Drop Lowest #20

Closed BuddyEthridge closed 12 years ago

BuddyEthridge commented 12 years ago

Let's discuss reimplementing Drop Lowest.

To reincorporate it, two things need to change:

Instructors will need to choose either Drop Lowest or Keep Highest (not both). This might already be the case; I cannot test it because we have Drop Lowest disabled.

Drop Lowest will need a limiter:

(X = graded items in category : Y = # of desired drops)

Do not drop grades until either X > Y or # of grades are present (where is manually set by the instructor)

Not sure if either of these limiters are doable, but thought I'd throw it on the table for discussion.

philcali commented 12 years ago

I've done some investigation on this with the current codebase.

Right now, only one option can be chosen. The form disables one if the other is chosen. So that works as expected.

For continued discussion, is it important to note the following when the system decides to exclude a grade via drop lowest or keep highest:

I was looking at what would be required to meet this requirement:

Do not drop grades until X > Y

It would not be difficult to achieve this from a coding perspective. My concern is that it adds a layer of complexity by being inconsistent.

rrusso commented 12 years ago

Agreed about the inconsistency.

What we need to do is add this as an administrative option (again) and explain why this option is different.

Not a rush on this one.

philcali commented 12 years ago

Sounds good. Like our other features, I'll make a separate branch for ease of integration.