Skip to content

Fix MedicalNERRecognizer loading with YAML context#2140

Open
ultramancode wants to merge 1 commit into
data-privacy-stack:mainfrom
ultramancode:fix/medical-ner-yaml-context
Open

Fix MedicalNERRecognizer loading with YAML context#2140
ultramancode wants to merge 1 commit into
data-privacy-stack:mainfrom
ultramancode:fix/medical-ner-yaml-context

Conversation

@ultramancode

Copy link
Copy Markdown
Contributor

Change Description

MedicalNERRecognizer could not be loaded from a YAML recognizer config when it was configured with language-specific context in supported_languages.

This change adds the missing optional context parameter to MedicalNERRecognizer and forwards it to the parent class.

Checklist

  • I have reviewed the contribution guidelines
  • I agree to follow this project's Code of Conduct
  • I confirm that I have the right to submit this contribution and that it does not knowingly contain proprietary or confidential code.
  • My code includes unit tests
  • All unit tests and lint checks pass locally
  • My PR contains documentation updates / additions if required

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant