Skip to content

Conversation

@BryanCutler
Copy link
Member

No description provided.

@BryanCutler
Copy link
Member Author

cc @frreiss , noticed this wasn't enabled in testing before and there as an import issue. I will make the release after this is merged.

from ibm_watson import NaturalLanguageUnderstandingV1
from ibm_cloud_sdk_core.authenticators import IAMAuthenticator
from ibm_watson.natural_language_understanding_v1 import Features, CategoriesOptions, ConceptsOptions, EmotionOptions, EntitiesOptions, KeywordsOptions, \
MetadataOptions, RelationsOptions, SemanticRolesOptions, SentimentOptions, SyntaxOptions, SyntaxOptionsTokens
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MetadataOptions was invalid, must have been from a very old version.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The API reference still lists it as valid, I'll open an issue
https://cloud.ibm.com/apidocs/natural-language-understanding?code=python#metadata

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

@frreiss frreiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BryanCutler BryanCutler merged commit cb821c2 into CODAIT:master Feb 1, 2021
@BryanCutler BryanCutler deleted the watson-remove-invalid-import-tests branch February 1, 2021 23:03
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.

2 participants