search

pivot-libre / tideman

Implementation of the Tideman ranked pairs algorithm
Apache License 2.0
9 stars 3 forks source link

Add Logging #45

Open carlschroedl opened 7 years ago

carlschroedl commented 7 years ago

As an election administrator I need a record of voters' rankings and the aggregate collective ranking so that I can satisfy open records requests.

One way to satisfy this is to add some logging. Minimally, we need to log the NBallots that are passed to RankedPairsCalculator.calculate(), and the CandidateList that is returned.

To satisfy this use case via logging, the deployment's logging configuration would need to be of appropriate verbosity to capture the messages. Concern for this could be obviated (aka hacked) by using a very high logging level like INFO(a bit much) or ERROR(clearly not appropriate in the long term).

andrewshell commented 7 years ago

I'll see if I can get this setup tonight.

carlschroedl commented 7 years ago

I made a first pass in this pull request https://github.com/pivot-libre/tideman/pull/47

carlschroedl commented 7 years ago

I wasn't sure whether I should sign the issue to you in addition to asking for review on the pull request

carlschroedl commented 7 years ago

We've merged some logging infrastructure. We have not yet added logging statements.