Skip to content

feat(NLC): Added classify_collection - #424

Merged
germanattanasio merged 1 commit into
developfrom
codegen/natural-language-classifier
Mar 28, 2018
Merged

feat(NLC): Added classify_collection#424
germanattanasio merged 1 commit into
developfrom
codegen/natural-language-classifier

Conversation

@ehdsouza

Copy link
Copy Markdown
Contributor
  • Regenerated NLC which added classify_collection
  • Added unit tests and example

@codecov-io

codecov-io commented Mar 28, 2018

Copy link
Copy Markdown

Codecov Report

Merging #424 into develop will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           develop    #424   +/-   ##
=======================================
  Coverage        0%      0%           
=======================================
  Files           22      22           
  Lines        12975   13038   +63     
=======================================
- Misses       12975   13038   +63
Impacted Files Coverage Δ
..._developer_cloud/natural_language_classifier_v1.py 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc88dd8...b6602f3. Read the comment docs.

natural_language_classifier.classify_collection(classifier_id, collection)

assert responses.calls[0].request.url == classify_collection_url
assert responses.calls[0].response.text == classify_collection_response

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please validate the request body

@germanattanasio
germanattanasio merged commit fa63cc9 into develop Mar 28, 2018
@germanattanasio
germanattanasio deleted the codegen/natural-language-classifier branch March 28, 2018 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants