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.

Sign up for GitHub

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

Jump to bottom

make MASC reader test portable #648

Closed

mssammon opened this issue May 17, 2018 · 1 comment

Contributor

mssammon commented May 17, 2018

Currently MASCXCESReaderTest fails if it is run on a non-CCG machine due to a hard-coded path that points outside the project:

https://github.com/CogComp/cogcomp-nlp/blob/master/corpusreaders/src/test/java/edu/illinois/cs/cogcomp/nlp/corpusreaders/mascReaderTests/MascXCESReaderTest.java#L27

Please copy a small subset of MASC XCES files to test/resources/ and point the test there.

lxtbell mentioned this issue

Add MascXCESReaderTest corpus to resources #650

Merged

Contributor Author

mssammon commented May 27, 2018

resolved by #650

mssammon closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment