[sakai2-tcc] [Building Sakai] New Feature: Assignment Peer Review SAK-23812

[Keli Sato Amann]

Hi Bryan.
 I'm pretty sure that this, along with group grading when it's ready, will be pretty popular on our campus but with other work going on this fall, not sure when we'll get to it. First though, we should probably unstealth Assignments--yes, believe it or not, we've gone this long without it--hopefully by Winter.

I had a few comments, more for long term fixes, no showstoppers 
1) students don't seem to have an indication of how many reviews they will have to do and may not know that they have to keep clicking "Next" until they can't. For instructors, is there anyway for them to see that a given student has done all the reviews they should and when they haven't, easily contact them to prod them along?
2) If a student can upload a file for anonymous grading, won't it potentially not be anonymous, because of metadata on file? Is there anyway to avoid this (strip metadata on upload, allow only inline submission, simply warn student)? Same issues for Samigo, btw, which is why I'm thinking of it.
3) students get a chance to review after they have turned in their work and after the due date. What happens if a student turns in their work late? Whose queue do they get added to and how will their grader know their work needs grading if they've already finished grading other's work?

Keli

----- Original Message -----
From: "Bryan Holladay" <holladay at longsight.com>
To: "Matthew Jones" <matthew at longsight.com>
Cc: "Sakai Development (sakai-dev at collab.sakaiproject.org)" <sakai-dev at collab.sakaiproject.org>, "sakai2-tcc Committee" <sakai2-tcc at collab.sakaiproject.org>
Sent: Friday, August 2, 2013 8:10:32 AM
Subject: Re: [Building Sakai] New Feature: Assignment Peer Review SAK-23812



I've set the property to "true" for all QA servers @r128109 



On Fri, Aug 2, 2013 at 11:02 AM, Matthew Jones < matthew at longsight.com > wrote: 



We talked internally, there really isn't a clear cut process, but generally I think all new features should be property protected even in trunk, at least the UI piece. 


This should be disabled by default, then we can enable the property on the nightly servers so it's visible and can be tested. If all testing goes well then TCC can vote prior to release to turn these features on by default. (or possibly even remove the property) 


-Matthew 





On Fri, Aug 2, 2013 at 10:54 AM, Bryan Holladay < holladay at longsight.com > wrote: 



Thanks for the feedback. I've committed this to trunk. 


One last question: 


Should this feature only be available through a Sakai.Property, which is current set to False by default? Should I set it to "True" by default or just remove it? 


-Bryan 





On Wed, Jul 31, 2013 at 3:05 PM, Alex Balleste < alexandre.balleste at udl.cat > wrote: 




It looks great. We will try it. 

Thanks for that new feature. 

Alex 




On 07/30/2013 09:04 PM, Bryan Holladay wrote: 





I have completed a new feature sponsored by NYU to allow Peer Reviews in the Assignments tool and I'd like to get this code into trunk. I've talked with Zhen and she will be out on vacation for the next two weeks so she said to go to the list. I've linked the documentation [1] and screen casts [2][3] as well as the Jira [4]. It's currently setup as a property feature (off by default), but I'd prefer no property mainly because properties tend to stay off and not get tested properly (not a deal breaker for me). A patch for 2.10 (4.0? 10.0?) and 2.9.x is attached to the jira. Derek Ramsey has spent a lot of time QA'ing this feature. 


I'm looking for a vote at a minimum and if someone wants to review the patch, that'd be swell. 


+1 from me :) 


Thanks, 
Bryan 





[1] https://docs.google.com/document/d/1KMoIOGjT9-1d1jc8LTIx6-75BQb95qjYJymhcKFXkSY/pub 
[2] Video Part 1: http://screencast.com/t/SX8zjMOky 
[3] Video Part 2: http://screencast.com/t/tsC3bHtn 

[4] https://jira.sakaiproject.org/browse/SAK-23812 


_______________________________________________
sakai-dev mailing list sakai-dev at collab.sakaiproject.org http://collab.sakaiproject.org/mailman/listinfo/sakai-dev TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe" 

-- 
Alexandre Ballesté Crevillén  alexandre.balleste at udl.cat =====================
University of Lleida
Information and Communication Systems Service
Analyst / Programmer
Twitter: @sinmsinm

Tlf: +34 973 702148 Fax: +34 973 702130 ===================== 
_______________________________________________ 
sakai-dev mailing list 
sakai-dev at collab.sakaiproject.org 
http://collab.sakaiproject.org/mailman/listinfo/sakai-dev 

TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe" 


_______________________________________________ 
sakai-dev mailing list 
sakai-dev at collab.sakaiproject.org 
http://collab.sakaiproject.org/mailman/listinfo/sakai-dev 

TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe" 



_______________________________________________
sakai-dev mailing list
sakai-dev at collab.sakaiproject.org
http://collab.sakaiproject.org/mailman/listinfo/sakai-dev

TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of "unsubscribe"