diff size problem when making changes

31 views
Skip to first unread message

Sarah Spall

unread,
Mar 20, 2013, 11:18:44 PM3/20/13
to utah-compiler...@googlegroups.com
Hi, my partner Graham and I ran the grading script on our assignment 2 code and passed 51 of the tests and failed 5.  We made a single change and moved one expression to the next line we expected the diff to be 2. The diff was 97 on the upside we now pasted 55 tests instead of 51. We followed the advice of some other people in the class and deleted the backup file because we were using emacs. Now our diff is at 300 and we are still passing the same amount of tests. 

Clearly we are doing something very wrong has anyone else encountered this problem?

Srikanth Raju

unread,
Mar 20, 2013, 11:23:12 PM3/20/13
to Sarah Spall, utah-compiler...@googlegroups.com
Did you look at the generated .diff file in ~/p2.grade/p2.diff to see what exactly has changed?

On Mar 20, 2013, at 9:18 PM, Sarah Spall <s.s...@utah.edu> wrote:

Hi, my partner Graham and I ran the grading script on our assignment 2 code and passed 51 of the tests and failed 5.  We made a single change and moved one expression to the next line we expected the diff to be 2. The diff was 97 on the upside we now pasted 55 tests instead of 51. We followed the advice of some other people in the class and deleted the backup file because we were using emacs. Now our diff is at 300 and we are still passing the same amount of tests. 

Clearly we are doing something very wrong has anyone else encountered this problem?

--
You received this message because you are subscribed to the Google Groups "Utah Compilers, Spring 2013" group.
To unsubscribe from this group and stop receiving emails from it, send an email to utah-compilers-spri...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
 
 

Parker Wightman

unread,
Mar 20, 2013, 11:28:39 PM3/20/13
to utah-compiler...@googlegroups.com
I ran into this same problem. I just rm'd my pyparse directory copied p2.turnin as pyparse, to start fresh. Then made EXACTLY the changes I needed to make (preferably not with emacs :-) ) and ended up getting a decent diff size. However, remember that if you're moving a line down several lines, that all lines in between will count toward your diff... I think. Anyway. Hope that helps.

Parker

Matt Might

unread,
Mar 20, 2013, 11:37:02 PM3/20/13
to Parker Wightman, utah-compiler...@googlegroups.com
In case others are having a similar problem:

If you only changed a few lines, but the diff exploded anyway,
document that in your README (including line numbers and the changes)
and send me an email with your uNID.

-Matt

Graham Pedersen

unread,
Mar 21, 2013, 10:20:20 PM3/21/13
to utah-compiler...@googlegroups.com, Parker Wightman, mi...@cs.utah.edu
Oh I forgot to post a status update.

We fixed the problem and now pass all the tests with a diff size of 10 or so. 
Reply all
Reply to author
Forward
0 new messages