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.

Sign up for GitHub

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

ftx namespace changes to any wrapper that uses it #87

Open
Piyush13y opened this issue Feb 20, 2022 · 1 comment
Open

ftx namespace changes to any wrapper that uses it #87

Piyush13y opened this issue Feb 20, 2022 · 1 comment
Assignees
@Piyush13y

Comments

@Piyush13y
Copy link
Collaborator

Piyush13y commented Feb 20, 2022 •
edited
Loading

Describe the bug
[To be resolved after PR goes through in Forte]
Space processor uses ftx.medical ontology, which needs to be updated to ftx.onto.clinical given our new namespacing changes in forte. In reference to this issue and comments from this PR,

To Reproduce
Just running any example using SpacyProcessor would throw an ImportError

Expected behavior
Import statement throws an error if the namespacing changes go through in forte and this change isn't made.
@Piyush13y Piyush13y changed the title Ensuring Ontology namespace consistency ftx namespace changes to any wrapper that uses it Feb 20, 2022
@Piyush13y
Copy link
Collaborator Author

@hunterhector Can you assign this to me, since I am unable to assign it to myself in this repo? Thanks!

This was referenced Feb 23, 2022
Closed
Closed
@hunterhector hunterhector mentioned this issue Mar 1, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Piyush13y Piyush13y

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ftx namespace changes to processors Piyush13y/forte-wrappers
1 participant
@Piyush13y