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

Ftx namespace changes to processors #89

Closed
wants to merge 2 commits into from

Conversation

Piyush13y
Copy link
Collaborator

This PR fixes #87

Description of changes

With reference to changes merged through Forte in PR, we updated the ontology namespace structure. The processors in Forte-wrapper needed to incorporate those updated namespaces. Only SpacyProcessor works with the ftx folder, hence that's the only file that has been updated.

Possible influences of this PR.

Code shouldn't throw Import errors now.

@hunterhector
Copy link
Member

hunterhector commented Mar 2, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ftx namespace changes to any wrapper that uses it
2 participants