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
python3Packages.somajo: init at 2.0.4 #81279
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the python38 tests seem to hang, not sure if upstream is aware
Strange! Works without issues on my machine (with sandboxing). |
hmm, the other two completed in 1-2 minutes, killed python38 after 10 |
@GrahamcOfBorg build python3Packages.somajo python38Packages.somajo |
if ofborg can build it, then I'm okay with it |
Updated to 2.0.4. @jonringer , this version fixes a race condition tsproisl/SoMaJo@fa67aa7 , maybe this was the hang that you ran into? |
could have been |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
commit LGTM
has test 👍
[4 built, 1 copied (1.3 MiB), 0.3 MiB DL]
https://github.com/NixOS/nixpkgs/pull/81279
2 package built:
python37Packages.somajo python38Packages.somajo
@GrahamcOfBorg build python37Packages.somajo python38Packages.somajo |
Motivation for this change
SoMaJo is a tokenizer for German and English web text and is quite popular in the German NLP community. Including this package is very low-impact, since it only relies on the Python
regex
package.The sources are retrieved from GitHub (rather than PyPI), so that we can run its test suite.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)