search

UNLV-ShaneD / unlvCS473kilo

5 stars 4 forks source link

Finished Charting. #23

Closed dabinett closed 12 years ago

dabinett commented 12 years ago

Something to point out. There is an error on Line 95 on ChartingController.java due to me not having Dane's stuff.

On lines 98-107, there is a test case for the graph. It currently displays as it should. I don't know if it works with Dane's stuff though. You may or may not want to leave it in for the presentation.

UNLV-ShaneD commented 12 years ago

It is past the deadline. I'm not manually fixing pull requests anymore - I have two exams to study for and I'm already losing sleep. I've already taken a hit on my first two exams for these same classes that I failed to study for because I was working on a project for this class.

You should probably learn how to resolve merge conflicts anyway - couldn't hurt. Merge the main repository into yours - it gets pretty much the same result. Then do a pull request.

dabinett commented 12 years ago

Calling projection doesn't work still, something breaks in Projection.java.. probably because there are no items. Thus, I left my test case in there so a graph still shows.