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

#1586 - Drop ClearNLP module #1601

Merged
merged 3 commits into from
Aug 8, 2024
Merged

Conversation

reckart
Copy link
Member

@reckart reckart commented Aug 7, 2024

What's in the PR

  • Remove the module
  • Updated Web1T to use the OpenNlpLemmatizer
  • Cleaning up

How to test manually

  • No specific test procedure

Automatic testing

  • PR includes unit tests

Documentation

  • PR updates documentation

reckart added 2 commits August 7, 2024 20:38
- Removed module
- Switch out lemmatizer in Web1T test
@reckart reckart added this to the 3.0.0 milestone Aug 7, 2024
@reckart reckart self-assigned this Aug 7, 2024
* main:
  #1599 - Add German lemmatizer model for OpenNLP

% Conflicts:
%	dkpro-core-opennlp-asl/src/scripts/build.xml
@reckart reckart force-pushed the refactoring/1586-Drop-ClearNLP-module branch from 40b7caf to d66cdf6 Compare August 8, 2024 00:02
@reckart reckart merged commit b800fcc into main Aug 8, 2024
3 checks passed
@reckart reckart deleted the refactoring/1586-Drop-ClearNLP-module branch August 8, 2024 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant