Skip to content

/ aut

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

Jump to bottom

Create tests for NERCombinedJson.scala #53

Closed
ruebot opened this issue Oct 2, 2017 · 1 comment
Closed

Create tests for NERCombinedJson.scala #53

ruebot opened this issue Oct 2, 2017 · 1 comment
Labels
RA-Task tests
Projects
Test coverage 1.0.0 Release of AUT

Comments

@ruebot
Copy link
Member

@ruebot ruebot commented Oct 2, 2017 •
edited

NERCombinedJson.scala has no test coverage. We need to create tests for it.
@ruebot ruebot added the tests label Oct 2, 2017
@ruebot ruebot added this to In Progress in Test coverage Oct 2, 2017
@ruebot ruebot moved this from In Progress to To Do in Test coverage Oct 2, 2017
@greebie
Copy link
Contributor

@greebie greebie commented Oct 19, 2017 •
edited

NER functions require a classifier file that is not included in AUT. Potential solutions:

  • develop a generic NER classifier for test purposes. See #48 & #52.
@greebie greebie mentioned this issue Oct 19, 2017
Closed
@ruebot ruebot added the RA-Task label Feb 5, 2018
@ruebot ruebot added this to In Progress in 1.0.0 Release of AUT May 27, 2020
ruebot added a commit that referenced this issue May 27, 2020
@ruebot
Remove NER functionality.
Verified
This commit was signed with a verified signature.
ruebot Nick Ruest
GPG key ID: 417FAF1A0E1080CD Learn about signing commits
Loading status checks…
624c68c 
- Resolves #48
- Resolves #52
- Resolves #53
- Resolves #469
- Remove all NER associated functionality
- Tweak pom.xml to handle the removal
@ruebot ruebot mentioned this issue May 27, 2020
Merged
1.0.0 Release of AUT automation moved this from In Progress to Done May 27, 2020
ianmilligan1 pushed a commit that referenced this issue May 27, 2020
@ruebot
Remove NER functionality. (#470)
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
060622c 
- Resolves #48
- Resolves #52
- Resolves #53
- Resolves #469
- Remove all NER associated functionality
- Tweak pom.xml to handle the removal
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
RA-Task tests
Projects
Test coverage
To Do
1.0.0 Release of AUT
  
Done
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

Remove NER functionality.
3 participants
@greebie @ruebot and others
You can’t perform that action at this time.