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: update tutorials avoid argilla installation error #337

Conversation

sdiazlor
Copy link
Contributor

@sdiazlor sdiazlor commented Feb 8, 2024

  • Moved the pip installation of argilla to the top to avoid importing errors later.
  • Some unneeded output removal.
  • Some updates of the sections on argilla

@sdiazlor sdiazlor linked an issue Feb 8, 2024 that may be closed by this pull request
@sdiazlor sdiazlor self-assigned this Feb 8, 2024
@davidberenstein1957
Copy link
Member

@sdiazlor, loooking good. I think we can just use one pip install [*, argilla] call. I think I saw pip install [openai] somewhere

@davidberenstein1957 davidberenstein1957 merged commit a701e83 into main Feb 8, 2024
4 checks passed
@davidberenstein1957 davidberenstein1957 deleted the docs/335-docs-update-tutorials-to-avoid-argilla-installation-error branch February 8, 2024 16:37
jphme pushed a commit to jphme/distilabel that referenced this pull request Feb 20, 2024
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.

[DOCS] update tutorials to avoid argilla installation error
2 participants