Differences between revisions 2 and 10 (spanning 8 versions)
⇤ ← Revision 2 as of 2007-04-09 18:20:17 →
Size: 1077
Editor: fw
Comment:
← Revision 10 as of 2007-04-09 22:32:46 → ⇥
Size: 2948
Editor: yakko
Comment:
Deletions are marked like this. Additions are marked like this.
Line 1: Line 1:
This is remarks for grading the project: Everyone except Hodges implemented the Gap-Graph algorithm for SAT p 175 in the book. This works great for Difference constraints, but not for Addition constraints. However everyone who turned that in did it right.

Hodges has some problems and I'd like to talk to him(I think its a him) about it. I gave him an 18because it appears like he tried to do the right algorithm. If he can show me where I need to change my tests to suit his solution I'll give him a 20. Yue went above and beyond with his interface. It looks really nice! Even though he only implemented difference constraints I gave him an 18.5 (90%). Everyone else got 18. If you want to bump them down by 1 than please bump Yue down .5 points.

Of course for those that didn't turn it in... 0.
Line 5: Line 9:
Apparently implemented <= instead of >=. Consequently one of my simple tests does not run. However it appears that he implemented the correct procedure and other than the above it look good. Apparently you implemented <= instead of >=. This should not be a problem, however the satisfiable tests fail. It appears that you implemented the correct procedure (SAT for Addition constraints not just difference constraints) and other than the above it look good. I would like to see you about a possible better grade. If you can show me that you can modify my tests to pass, I'll give you a 20/20.
Line 7: Line 11:
19    * testSimpleSat: Failed
   * testSimpleUnsat: Passed
   * testDiffUnsat: Passed
   * testDiff2Sat: Failed
   * testAddUnsat: Passed
   * testAddSat: Failed

3/6*

Recommend: 18
Line 11: Line 24:
Implemented the book sat method which only does difference constraints. You only implemented difference constraints.
Line 13: Line 26:
17    * testSimpleSat: Passed
   * testSimpleUnsat: Passed
   * testDiffUnsat: Passed
   * testDiff2Sat: Passed
   * testAddUnsat: Failed
   * testAddSat: Failed

4/6

Recommend: 18
Line 18: Line 40:
You only implemented difference constraints. But a very nice program interface! Passes all difference tests. Fails x+y>=b tests and -x-y>=b tests. You only implemented difference constraints. But a very nice program interface!
Line 20: Line 42:
18    * testSimpleSat: Passed
   * testSimpleUnsat: Passed
   * testDiffUnsat: Passed
   * testDiff2Sat: Passed
   * testAddUnsat: Failed
   * testAddSat: Failed

4/6

Recommend: 18.5
Line 24: Line 55:
You only implemented difference constraints. Passes all difference tests. Fails x+y>=b tests and -x-y>=b tests. You only implemented difference constraints.
Line 26: Line 57:
18    * testSimpleSat: Passed
   * testSimpleUnsat: Passed
   * testDiffUnsat: Passed
   * testDiff2Sat: Passed
   * testAddUnsat: Failed
   * testAddSat: Failed

4/6

Recommend: 18
Line 30: Line 70:
You only implemented difference constraints. Passes all difference tests. Fails x+y>=b tests and -x-y>=b tests. You only implemented difference constraints.
Line 32: Line 72:
18    * testSimpleSat: Passed
   * testSimpleUnsat: Passed
   * testDiffUnsat: Passed
   * testDiff2Sat: Passed
   * testAddUnsat: Failed
   * testAddSat: Failed

4/6

Recommend: 18
Line 36: Line 85:
You only implemented difference constraints. Passes all difference tests. Fails x+y>=b tests and -x-y>=b tests. You only implemented difference constraints.
Line 38: Line 87:
18    * testSimpleSat: Passed
   * testSimpleUnsat: Passed
   * testDiffUnsat: Passed
   * testDiff2Sat: Passed
   * testAddUnsat: Failed
   * testAddSat: Failed

4/6

Recommend: 18
Line 42: Line 100:
You only implemented difference constraints. Passes all difference tests. Fails x+y>=b tests and -x-y>=b tests. You only implemented difference constraints.
Line 44: Line 102:
18    * testSimpleSat: Passed
   * testSimpleUnsat: Passed
   * testDiffUnsat: Passed
   * testDiff2Sat: Passed
   * testAddUnsat: Failed
   * testAddSat: Failed

4/6

Recommend: 18

Everyone except Hodges implemented the Gap-Graph algorithm for SAT p 175 in the book. This works great for Difference constraints, but not for Addition constraints. However everyone who turned that in did it right.

Hodges has some problems and I'd like to talk to him(I think its a him) about it. I gave him an 18because it appears like he tried to do the right algorithm. If he can show me where I need to change my tests to suit his solution I'll give him a 20. Yue went above and beyond with his interface. It looks really nice! Even though he only implemented difference constraints I gave him an 18.5 (90%). Everyone else got 18. If you want to bump them down by 1 than please bump Yue down .5 points.

Of course for those that didn't turn it in... 0.

Hodges

Apparently you implemented <= instead of >=. This should not be a problem, however the satisfiable tests fail. It appears that you implemented the correct procedure (SAT for Addition constraints not just difference constraints) and other than the above it look good. I would like to see you about a possible better grade. If you can show me that you can modify my tests to pass, I'll give you a 20/20.

  • testSimpleSat: Failed
  • testSimpleUnsat: Passed
  • testDiffUnsat: Passed
  • testDiff2Sat: Failed
  • testAddUnsat: Passed
  • testAddSat: Failed

3/6*

Recommend: 18

Augustyn

You only implemented difference constraints.

  • testSimpleSat: Passed
  • testSimpleUnsat: Passed
  • testDiffUnsat: Passed
  • testDiff2Sat: Passed
  • testAddUnsat: Failed
  • testAddSat: Failed

4/6

Recommend: 18

Yue

You only implemented difference constraints. But a very nice program interface!

  • testSimpleSat: Passed
  • testSimpleUnsat: Passed
  • testDiffUnsat: Passed
  • testDiff2Sat: Passed
  • testAddUnsat: Failed
  • testAddSat: Failed

4/6

Recommend: 18.5

Sadofsky

You only implemented difference constraints.

  • testSimpleSat: Passed
  • testSimpleUnsat: Passed
  • testDiffUnsat: Passed
  • testDiff2Sat: Passed
  • testAddUnsat: Failed
  • testAddSat: Failed

4/6

Recommend: 18

Neo

You only implemented difference constraints.

  • testSimpleSat: Passed
  • testSimpleUnsat: Passed
  • testDiffUnsat: Passed
  • testDiff2Sat: Passed
  • testAddUnsat: Failed
  • testAddSat: Failed

4/6

Recommend: 18

Miao

You only implemented difference constraints.

  • testSimpleSat: Passed
  • testSimpleUnsat: Passed
  • testDiffUnsat: Passed
  • testDiff2Sat: Passed
  • testAddUnsat: Failed
  • testAddSat: Failed

4/6

Recommend: 18

Ramsey

You only implemented difference constraints.

  • testSimpleSat: Passed
  • testSimpleUnsat: Passed
  • testDiffUnsat: Passed
  • testDiff2Sat: Passed
  • testAddUnsat: Failed
  • testAddSat: Failed

4/6

Recommend: 18

Cse913ProjS107 (last edited 2007-04-09 22:32:46 by yakko)