Skip to content
This repository has been archived by the owner on Nov 29, 2017. It is now read-only.

Scoring System #10

Closed
wants to merge 10 commits into from
Closed

Scoring System #10

wants to merge 10 commits into from

Conversation

vapour101
Copy link
Owner

Should mostly work for Japanese scoring, I still want to add one or two test cases from real games, though

These files need to be shared by necessity so that everyones' code style is consistent
Added a new class "DrawCoords" to hold 2D coordinates for draw calls, this should increase the readability of the code.
Simple scoring now works and is tested
Added more tests and fixed an obscure bug that the tests revealed
@coveralls
Copy link

Coverage Status

Coverage increased (+6.5%) to 100.0% when pulling a676c40 on scoring into 3769b38 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+6.5%) to 100.0% when pulling e3946e5 on scoring into 3769b38 on master.

Added a real game to the BoardScorer testsuite, confirming that the scorer works as intended.
Yet another test from a real game, however scoring this one is far more complicated.
@coveralls
Copy link

Coverage Status

Coverage increased (+6.5%) to 100.0% when pulling 9c24a04 on scoring into 3769b38 on master.

@vapour101
Copy link
Owner Author

Pulled this into #11, so I'm closing it.

@vapour101 vapour101 closed this Aug 20, 2017
@vapour101 vapour101 deleted the scoring branch August 20, 2017 20:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants