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

adding new parameter to nlp learner #235

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pedroschoen
Copy link

Instructions

  • Follow the instructions in README.md
  • Delete everything between parenthesis (...)
  • Remove the sections that are not relevant

Status

READY
IN DEVELOPMENT
HOLD
(leave only one of the options above)

  • IN DEVELOPMENT: there's active development being done in this branch, don't merge yet
  • HOLD: development is done, but PR shouldn't be merged yet (e.g. a correspondent PR in another service needs to be merged first)
  • READY: development over and everything ready for merging
    (leave only one of the options above)

Todo list

  • Documentation
  • Tests added and passed
  • Issue: closes #xxxx (write the number of the GitHub issues that this PR resolves)

Background context

(describe the context, motivation, problem)

Description of the changes proposed in the pull request

(a few sentences describing the overall goals of the pull request's commits)

Related PRs

(list of related PRs. Use the GitHub numbering.)

Where should the reviewer start?

(describe where the reviewer should start reviewing the PR)

Remaining problems or questions

(write the remaining problems or questions that this PR does not solve)

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.

1 participant