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

Throw proper error message for configuration type mismatch #79

Open
jennyzhang-petuum opened this issue Nov 5, 2021 · 0 comments
Open

Comments

@jennyzhang-petuum
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
When running SpacyProcessor, the config field processor need to be a list of string and it didn't throw error when processor field was provided a string but this string would not work as this config field. Because it does an iteration of the config in the set up part, if the config is a string then it simply loop each character.

Describe the solution you'd like
To do a check on the Type of the config field processor when read in. It could be done in the function validate_spacy_configs

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

No branches or pull requests

1 participant