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

fix/loss video updated #853

Merged
merged 2 commits into from
Dec 28, 2021

Conversation

leireropl
Copy link
Contributor

Please. leave feedback on the video

@dvsrepo
Copy link
Member

dvsrepo commented Dec 27, 2021

Hi Leire,

In general I like the video, but if possible I would recommend the following:

Start the video with no pre-validated annotations. I guess you were doing tests before. Either you delete and load again the dataset or deselect the labels. Once you start the video you should: (1) sort by desc loss, (2) start correcting the labels sequentially from the first record, no need to go to the next page. Another option to do this more sequential would be (1) sort, (2) select 1 record per page, (3) start correcting labels and use the right keyboard arrow (->) to go to the next record.

@leireropl
Copy link
Contributor Author

leireropl commented Dec 28, 2021

Hi Leire,

In general I like the video, but if possible I would recommend the following:

Start the video with no pre-validated annotations. I guess you were doing tests before. Either you delete and load again the dataset or deselect the labels. Once you start the video you should: (1) sort by desc loss, (2) start correcting the labels sequentially from the first record, no need to go to the next page. Another option to do this more sequential would be (1) sort, (2) select 1 record per page, (3) start correcting labels and use the right keyboard arrow (->) to go to the next record.

Hi Dani,

Thanks for the tip. I have just uploaded a new one, let me know what you think.

@dvsrepo
Copy link
Member

dvsrepo commented Dec 28, 2021

it looks perfect!

@dvsrepo dvsrepo merged commit 62f1a49 into argilla-io:master Dec 28, 2021
dvsrepo added a commit that referenced this pull request Dec 28, 2021
* 'master' of https://github.com/recognai/rubrix:
  chore(ci): Include component testing configuration (#839)
  fix/loss video updated (#853)
  Docs: Weak supervision guide update (#855)
frascuchon pushed a commit that referenced this pull request Jan 11, 2022
* fix/loss video updated
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.

None yet

2 participants