-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Agg vs gold fixes #70
Conversation
Closes #49 |
This now tests gold standard vs. the aggregation RData output with all of the ISO code fixes in place (i.e., it behaves as if this PR and #68 have both been merged already). With those ISO code fixes in place, the agg vs. gold disparity is down to 4 languages present in gold but not in agg, and 15 inventories with discrepancies between agg & gold. |
…agg-vs-gold-fixes
alleged glottal tap -> glottal stop. closes phoible#83
fix: representation of clicks in nmn from GM
…easier rebasing of agg-vs-gold-fixes branch. This reverts commit 701ff69.
…agg-vs-gold-fixes. pull from origin after rebase against upstream master.
@bambooforest, it's probably time to merge this before things get any more tangled. There are still a few agg vs gold differences, but they'll be easier to see once this is merged and I can look at it afresh. |
I agree. Will review and merge. |
This adds a test script to compare the "gold standard" database dump with the output of the aggregation script, and fixes some of the problems that it reveals (though not yet all). Should not be merged yet, until we figure out what to do about the remaning discrepancies between
agg
andgold
. Note that most of the remaining discrepancies are due to the UPSID "dental|alveolar" agnosticism.You can test this locally with
git checkout -b testAggGold drammock/agg-vs-gold-fixes
(assuming you have a remote called
drammock
that points todrammock/phoible
repo).