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

UI : code given in the Argilla starting page for logging data has "y" which causing invalid syntax #1811

Closed
Jitendra-Dash opened this issue Nov 2, 2022 · 3 comments · Fixed by #1814
Assignees
Labels
type: bug Indicates an unexpected problem or unintended behavior
Milestone

Comments

@Jitendra-Dash
Copy link

Jitendra-Dash commented Nov 2, 2022

Describe the bug
Before creating a dataset, a code is provided to log data into argilla using python in the argilla UI. In that code, there is a "y" letter which is of no use and throws error.

To Reproduce
Steps to reproduce the behavior:

  1. Go to the 'Argilla Start page without any dataset'
  2. you will see a code for logging data using python
  3. use the same code

Expected behavior
Invalid syntax

Screenshots
If applicable, add screenshots to help explain your problem.

Environment (please complete the following information):

  • OS [e.g. iOS]:
  • Browser [e.g. chrome, safari]:
  • Argilla Version [e.g. 1.0.0]:
  • ElasticSearch Version [e.g. 7.10.2]:
  • Docker Image (optional) [e.g. argilla:v1.0.0]:

Additional context
Add any other context about the problem here.

@Jitendra-Dash Jitendra-Dash added the type: bug Indicates an unexpected problem or unintended behavior label Nov 2, 2022
@Jitendra-Dash Jitendra-Dash changed the title code given in the Argilla starting page for logging data has "y" which causing invalid syntax UI : code given in the Argilla starting page for logging data has "y" which causing invalid syntax Nov 2, 2022
@dvsrepo
Copy link
Member

dvsrepo commented Nov 2, 2022

Thanks so much for reporting, this will be fixed ASAP

@dvsrepo dvsrepo assigned dvsrepo and leiyre and unassigned dvsrepo Nov 2, 2022
@dvsrepo
Copy link
Member

dvsrepo commented Nov 2, 2022

@leiyre this is the same as #1784 (closed in favour of this one)

I will take this opportunity to add a more meaningful example (#1801)

Replace the one we have by:

# install datasets library with pip install datasets
import argilla as rg
from datasets import load_dataset

# load dataset from the hub
dataset = load_dataset("argilla/gutenberg_spacy-ner", split="train")

# read in dataset, assuming its a dataset for token classification
dataset_rg = rg.read_datasets(dataset, task="TokenClassification")

# log the dataset
rg.log(dataset_rg, "gutenberg_spacy-ner")

leiyre added a commit that referenced this issue Nov 2, 2022
This PR updates the code and remove the extra letter "y"

closes #1811
@leiyre leiyre linked a pull request Nov 2, 2022 that will close this issue
@frascuchon frascuchon added this to the v1.0.1 milestone Nov 3, 2022
@leiyre
Copy link
Member

leiyre commented Nov 4, 2022

Hello @Jitendra-Dash, the issue has already been fixed, you can update to version 1.0.1.
https://github.com/argilla-io/argilla/releases/tag/v1.0.1,
Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants