Expertiza_Wiki - User contributions [en]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T04:09:23Z

Njdowdle: /* Files Requiring Modification */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

 
Shown below is a diagram of how the current system works (copying the extra participants/reviews):
[[File:Calibration2.png|600px|left]]
 
 
 

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''Testing Goals''' ===

The main goal of our testing is to ensure that when a new calibration assignment is added, all of the qualifying previous calibration submissions
and reviews are successfully copied over without overwriting any existing submissions.

=== '''Automated Unit Tests''' ===

Using RSpec, we have to test that the following unit tests pass:

1. When the TA adds a calibration assignment, extra participants are added to the assignment.

2. When extra participants are added to an assignment, calibration submissions are copied over.

3. When calibration submissions are copied over, the submitter_count value increases by the number of calibration submissions, so that previous submissions are
not overwritten.

4. After calibration assignments are copied over, only calibration reviews from the creator of the assignment are copied over.

5. After calibration assignments are copied over, only calibration reviews that have been started are copied over.

6. Calling specific methods in assignment.rb return a NoMethodError to indicate that these methods are no longer class methods and have been moved elsewhere as
instance methods.

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T04:08:33Z

Njdowdle: /* Previous Implementation */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

 
Shown below is a diagram of how the current system works (copying the extra participants/reviews):
[[File:Calibration2.png|600px|left]]
 
 
 

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''Testing Goals''' ===

The main goal of our testing is to ensure that when a new calibration assignment is added, all of the qualifying previous calibration submissions
and reviews are successfully copied over without overwriting any existing submissions.

=== '''Automated Unit Tests''' ===

Using RSpec, we have to test that the following unit tests pass:

1. When the TA adds a calibration assignment, extra participants are added to the assignment.

2. When extra participants are added to an assignment, calibration submissions are copied over.

3. When calibration submissions are copied over, the submitter_count value increases by the number of calibration submissions, so that previous submissions are
not overwritten.

4. After calibration assignments are copied over, only calibration reviews from the creator of the assignment are copied over.

5. After calibration assignments are copied over, only calibration reviews that have been started are copied over.

6. Calling specific methods in assignment.rb return a NoMethodError to indicate that these methods are no longer class methods and have been moved elsewhere as
instance methods.

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T04:07:06Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

 
Shown below is a diagram of how the current system works:
[[File:Calibration2.png|600px|left]]
 
 
 

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''Testing Goals''' ===

The main goal of our testing is to ensure that when a new calibration assignment is added, all of the qualifying previous calibration submissions
and reviews are successfully copied over without overwriting any existing submissions.

=== '''Automated Unit Tests''' ===

Using RSpec, we have to test that the following unit tests pass:

1. When the TA adds a calibration assignment, extra participants are added to the assignment.

2. When extra participants are added to an assignment, calibration submissions are copied over.

3. When calibration submissions are copied over, the submitter_count value increases by the number of calibration submissions, so that previous submissions are
not overwritten.

4. After calibration assignments are copied over, only calibration reviews from the creator of the assignment are copied over.

5. After calibration assignments are copied over, only calibration reviews that have been started are copied over.

6. Calling specific methods in assignment.rb return a NoMethodError to indicate that these methods are no longer class methods and have been moved elsewhere as
instance methods.

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T04:01:14Z

Njdowdle: /* Previous Implementation */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

 
Shown below is a diagram of how the current system works:
[[File:Calibration2.png|600px|left]]
 
 
 

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''Testing Goals''' ===

The main goal of our testing is to ensure that when a new calibration assignment is added, all of the qualifying previous calibration submissions
and reviews are successfully copied over without overwriting any existing submissions.

=== '''Automated Unit Tests''' ===

Using RSpec, we have to test that the following unit tests pass:

1. When the TA adds a calibration assignment, extra participants are added to the assignment.

2. When extra participants are added to an assignment, calibration submissions are copied over.

3. When calibration submissions are copied over, the submitter_count value increases by the number of calibration submissions, so that previous submissions are
not overwritten.

4. After calibration assignments are copied over, only calibration reviews from the creator of the assignment are copied over.

5. After calibration assignments are copied over, only calibration reviews that have been started are copied over.

6. Calling specific methods in assignment.rb return a NoMethodError to indicate that these methods are no longer class methods and have been moved elsewhere as
instance methods.

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T04:01:03Z

Njdowdle: /* Previous Implementation */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

 
Shown below is a diagram of how the current system works:
[[File:Calibration2.png|400px|left]]
 
 
 

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''Testing Goals''' ===

The main goal of our testing is to ensure that when a new calibration assignment is added, all of the qualifying previous calibration submissions
and reviews are successfully copied over without overwriting any existing submissions.

=== '''Automated Unit Tests''' ===

Using RSpec, we have to test that the following unit tests pass:

1. When the TA adds a calibration assignment, extra participants are added to the assignment.

2. When extra participants are added to an assignment, calibration submissions are copied over.

3. When calibration submissions are copied over, the submitter_count value increases by the number of calibration submissions, so that previous submissions are
not overwritten.

4. After calibration assignments are copied over, only calibration reviews from the creator of the assignment are copied over.

5. After calibration assignments are copied over, only calibration reviews that have been started are copied over.

6. Calling specific methods in assignment.rb return a NoMethodError to indicate that these methods are no longer class methods and have been moved elsewhere as
instance methods.

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

File:Calibration2.png

2022-04-12T04:00:43Z

Njdowdle:

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:45:06Z

Njdowdle: /* Previous Implementation */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

 
Shown below is a diagram of how the current system works:
[[File:Calibration.png|400px|left]]
 
 
 

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:44:56Z

Njdowdle: /* Problem Statement */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:38:29Z

Njdowdle: /* Problem Statement */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.
 
Shown below is a diagram of how the current system works:
[[File:Calibration.png|400px|left]]
 
 
 

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:38:17Z

Njdowdle: /* Problem Statement */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.
 
Shown below is a diagram of how the current system works:
[[File:Calibration.png|400px|left]]
 
 
 

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:38:07Z

Njdowdle: /* Problem Statement */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.
 
Shown below is a diagram of how the current system works:
[[File:Calibration.png|400px|left]]
 
 

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:37:53Z

Njdowdle: /* Problem Statement */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.
 
Shown below is a diagram of how the current system works:
[[File:Calibration.png|400px|left]]
 

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:37:28Z

Njdowdle: /* Problem Statement */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.
 
Shown below is a diagram of how the current system works:
[[File:Calibration.png|800px|left]]

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

File:Calibration.png

2022-04-12T03:35:46Z

Njdowdle:

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:07:59Z

Njdowdle: /* Files Requiring Modification */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write the create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:07:39Z

Njdowdle: /* Files Requiring Modification */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write this create method to create the extra participants.
** [[File:wiki2.png|600px|left]] 

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:06:54Z

Njdowdle: /* Files Requiring Modification */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and put this function in a new method. This code currently resides in the self.copy method (which is too long and does too much for one method), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write a create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:05:33Z

Njdowdle: /* Files Requiring Modification */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write a create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:05:20Z

Njdowdle: /* Files Requiring Modification */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb:
** [[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write a create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:04:50Z

Njdowdle: /* Files Requiring Modification */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb: 
[[File:wiki2.png|600px|left]] 
 
 
 
 
 
 
 
 
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write a create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:04:28Z

Njdowdle: /* Files Requiring Modification */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
** Here is the current create method in participant.rb: 
[[File:wiki2.png|800px|left]] 

* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write a create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

File:Wiki2.png

2022-04-12T03:03:57Z

Njdowdle: Njdowdle uploaded a new version of File:Wiki2.png

class c

File:Wiki2.png

2022-04-12T03:03:52Z

Njdowdle: Njdowdle uploaded a new version of File:Wiki2.png

class c

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T03:00:19Z

Njdowdle: /* Files Requiring Modification */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
**Intead of setting the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc, We will change it to name "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
* participant.rb
** Write the create method in assignment_participant instead of directly in participant.rb
* team.rb
** Create a copy method in team.rb that copies the field used by all teams
* assignment_team.rb
* assignment_participant.rb
** Write a create method to create the extra participants.

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:46:54Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:31:56Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:31:08Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:30:42Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Break out copy of new assignment name into a new method
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method (which it too lengthy), so we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:29:07Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Break out copy of new assignment name into a new method
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method, but we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:28:43Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Break out copy of new assignment name into a new method
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method, but we will move it to a new method called copy_name.
[[File:wiki1.png|800px|left]] 
 
 
 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:28:25Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Break out copy of new assignment name into a new method
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method, but we will move it to a new method called copy_name.
**[[File:wiki1.png|800px|left]] 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:27:58Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Break out copy of new assignment name into a new method
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method, but we will move it to a new method called copy_name.
[[File:wiki1.png|800px|left]] 
 
*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:27:39Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Break out copy of new assignment name into a new method
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method, but we will move it to a new method called copy_name.
[[File:wiki1.png|800px|left]] 

*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:26:36Z

Njdowdle: /* What needs to be done */

== '''Introduction''' ==
A “calibration assignment” is an assignment where the students are asked to review work that has also been reviewed by a member of the course staff. If the student’s review “resembles” the staff-member’s review, then the student is presumed to be a competent reviewer. Here is a further description of calibration assignments. To set up calibration, the instructor (or TA) adds a few extra participants to the assignment. The instructor (or TA) then impersonates the extra participants, and submits work on behalf of each of the extra participants.

== '''Problem Statement''' ==
Having the instructor (or TA) impersonate the extra participants, and submit work on behalf of each of the extra participants is obviously extra trouble. It would be nice if an instructor didn’t have to resubmit the same calibration submissions every semester.

== '''Previous Implementation''' ==
The previous project did fully implement code that copies over the calibration submissions whenever a calibrated assignment is submitted. It was done by ensuring that when a assignment is copied that 1) the same extra participants as the assignment you are copying are used 2) the URLs and/or files that were submitted to the previous assignment are also copied and 3) the URLs and files are submitted to the assignment as well

The previous implementation needs to be revisited and made more elegant. A description of the needed refactoring and changes is below.

=== '''What needs to be done''' ===
*Ensure what gets copied (when and how) is done properly
**Reviews done by the creator of the assignment are copied, but only reviews that have been started --> how do you recognize a calibration review?
**If an author did any non-calibration reviews in the assignment being copied, those reviews would also be treated as calibration reviews
**Submitted files are copied too, but if the calibration submissions are done as part of the COPYING of the assignment, they will necessarily use directories 0, 1,
*Handling the case of students submitting assignments before calibration assignments are done
**Project assumes that calibrations start at directory 0
**Not the case if any students submit before the calibration submissions are done
*Submitter_count needs to be set to the largest directory number that was used for calibration
**First student to submit will have their submission show up in the same directory with the first calibration submission
*Several class methods in assignment_form.rb should be moved to different classes as instance methods in those classes
**Add better comments - eg. a comment for a method that copies an assignment should not reveal details about calibrations. Currently, only 1/4 methods in this class have comments.
**Break out copy of new assignment name into a new method
**Currently, for assignment_form.rb we change the name of the copied assignments to “copy of copy” and "copy of copy of copy" etc. We will improve this by making it "Copy of <name> <copy number>" and putting this function in a new method. This code currently resides in the self.copy method, but we will move it to a new method called copy_name.
[[File:wiki1.png|800px|left]]

*Clean up participant.rb
**Look for a more elegant way of (self.create_participant) copying/creating of participants → DRY violation
**Write the create method in assignment_participant instead of directly in participant.rb
*Clean up team.rb
**Create a copy method in team that copies the field used by all teams
**Rest of the stuff is just used by assignment teams (submitted hyperlinks etc.)
**Create a copy method and override in assignment_team
*Reevaluate instance variables. Are these needed, or could they be local variables instead?
*Look at editing out 9min video due to long waits for responses during copying
*Verify our work using not only user 'Instructor6' but also Professor Jennifer Kidd's account - 'Instructor 2026'

== '''Design''' ==

==='''Files Requiring Modification'''===
* assignment_form.rb
* participant.rb
* team.rb
* assignment_team.rb
* assignment_participant.rb

== '''Final Implementation''' ==

== '''Test Plan''' ==

=== '''UI Testing''' ===

=== '''Automated Testing''' ===

== '''Conclusion ''' ==

== '''Team Information''' ==

Mentor: Dr. Ed Gehringer (efg@ncsu.edu) 
Nolan Dowdle (njdowdle@ncsu.edu) 
Laura Fox (lmcampb5@ncsu.edu) 
Soeun Jo (sjo@ncsu.edu) 
Harshil Shah (hshah6@ncsu.edu) 

== '''Resources''' ==
A link to the Expertiza GitHub can be found [https://github.com/expertiza/expertiza here] 
A link to our repository can be found [https://github.com/njdowdle/expertiza/tree/beta here] 
Our implementation video can be found here 
A link to the previous Wiki can be found [https://expertiza.csc.ncsu.edu/index.php/CSC/ECE_517_Fall_2020_-_E2075._calibration_submissions_should_be_copied_along_with_calibration_assignments#Project_Overview here] 
The previous implementation video can be found [https://youtu.be/cyhy2mVXcPM here] 
A link to the previous GitHub Repo can be found [https://github.com/expertiza/expertiza/pull/1856 here]

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:23:56Z

Njdowdle: /* What needs to be done */

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:17:49Z

Njdowdle: /* What needs to be done */

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:17:26Z

Njdowdle: /* What needs to be done */

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:15:28Z

Njdowdle: /* Previous Implementation */

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:15:10Z

Njdowdle: /* Previous Implementation */

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:14:36Z

Njdowdle: /* What needs to be done */

CSC/ECE 517 Spring 2022 - E2234. Calibration submissions should be copied along with calibration assignments

2022-04-12T02:12:47Z

Njdowdle: /* What needs to be done */

File:Wiki1.png

2022-04-12T02:10:50Z

Njdowdle: Njdowdle uploaded a new version of File:Wiki1.png

Modules