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

[DOCS] Typos in "Train a Model" Docs #3429

Closed
4 tasks
peppinob-ol opened this issue Jul 18, 2023 · 1 comment
Closed
4 tasks

[DOCS] Typos in "Train a Model" Docs #3429

peppinob-ol opened this issue Jul 18, 2023 · 1 comment
Assignees

Comments

@peppinob-ol
Copy link
Contributor

peppinob-ol commented Jul 18, 2023

Which page or section is this issue related to?

https://docs.argilla.io/en/latest/guides/train_a_model.html#the-argillatrainer

missing "e" in DatasetFor-

  • change DatastForTextClassification to DatasetForTextClassification
  • change DatastForTokenClassification to DatasetForTokenClassification
  • change DatastForTokenClassification to DatasetForTokenClassification

Traindirectly>Example Workflow
Missing "from argilla.training import ArgillaTrainer" in the code snippet

  • add from argilla.training import ArgillaTrainer in the code snippet
@tomaarsen tomaarsen self-assigned this Jul 24, 2023
alvarobartt pushed a commit that referenced this issue Jul 26, 2023
Closes #3429

Hello!

# Description

Resolves typos and a missing import as described by #3429

**Type of change**

- [x] Documentation update

**How Has This Been Tested**

I ran the snippet that I modified - it worked.

**Checklist**

- [x] I added relevant documentation
- [ ] follows the style guidelines of this project
- [x] I did a self-review of my code
- [ ] I made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I filled out [the contributor form](https://tally.so/r/n9XrxK)
(see text above)
- [ ] I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---

- Tom Aarsen
@nataliaElv
Copy link
Member

This issue was solved in PR #3443

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

3 participants