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

REPP versions #3

Open
arademaker opened this issue Aug 10, 2022 · 1 comment
Open

REPP versions #3

arademaker opened this issue Aug 10, 2022 · 1 comment
Labels
question Further information is requested

Comments

@arademaker
Copy link
Member

What is the relation between this code and the code from Woodley?

https://github.com/delph-in/homebrew-delphin/blob/HEAD/Formula/repp.rb#L4

I know from https://github.com/delph-in/docs/wiki/ReppTop that this code from Woodley and the @goodmami implementation at https://pydelphin.readthedocs.io/en/latest/api/delphin.repp.html are alternative implementations. Are all of these 100% compatible?

@goodmami
Copy link
Member

Woodley's version is what's used in ACE, and I believe it predates this implementation a little. This repo is the code used for PET and for the standalone repp command (which is currently used in the NLTK's nltk.tokenize.repp module). Two other implementations include PyDelphin and the LKB's (which probably should be listed in the ReppTop wiki's "Implementations" section, even though it's mentioned elsewhere in the doc).

They are mostly compatible. The main differences are masking support and characterization (start/stop indices of tokens). This repo and Woodley's repp-0.2.2 release do not include masking, but Woodley has an unreleased version of his implementation with masking support that is used in recent versions of ACE. The LKB and PyDelphin both have masking support. And where PyDelphin follows this repo's characterization behavior exactly, Woodley's code, last I checked, outputs different characterization in some cases. I don't recall what the LKB does for characterization.

@goodmami goodmami added the question Further information is requested label Aug 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants