Preserve dictionary order in the anonymizer.

Review Request #1781 - Created Feb. 16, 2015 and submitted

Information
Benjy Weinberger
pants
bd868f3...
Reviewers
pants-reviews
patricklaw
This matters because when splitting zinc analysis files
we must choose a representative class from each source file
(see that code for why), and in some cases we end up choosing
the first class in dictionary order. If the anonymized version
doesn't pick the equivalent representative, tests will fail.

Used this to generate the new test data for https://rbcommons.com/s/twitter/r/1779/. It was a failing test there that alerted me to this complication.

Issues

  • 0
  • 3
  • 0
  • 3
Description From Last Updated
Stu Hood
Benjy Weinberger
Patrick Lawson
Stu Hood
Patrick Lawson
Benjy Weinberger
Benjy Weinberger
Review request changed

Status: Closed (submitted)

Change Summary:

Thanks all. Submitted as e44aacaf80c34445c555bce9bb0f3e2e441cf3c7.

Loading...