Skip to content
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

Added flag to import conditional probability table #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dowobeha
Copy link
Contributor

Note: if you run an alignment and export the conditional probability table, then later import that table and use it to align your original parallel corpus, there may be differences in the final alignments. I assume this is due to floating point rounding errors introduced during the import/export cycle. I believe this because when you do the preceding, and then continue by again exporting the table, the newly-exported table exactly matches the originally-exported table. These discrepancies in final alignments could presumably be eliminated if the exported tables stored counts rather than probabilities. However, the ttable class currently clears counts when it performs normalization, so for now I'm going to leave things as they are.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant