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 next button to the labeling interface #12

Merged
merged 2 commits into from
Jul 25, 2024
Merged

Adding next button to the labeling interface #12

merged 2 commits into from
Jul 25, 2024

Conversation

dat-boris
Copy link
Collaborator

@dat-boris dat-boris commented Jul 25, 2024

Summary:

To make labelling easier, provide the Next task button to allow labellers to iterate through items.

Note that normally one would use the "Label tasks as displayed" feature, but that turns out to be
not working as expected - at most it will give label of ~10 tasks from our testing.

Note that we create a separate button, instead of part of the "Submit" because submitting require
an async operation to data manager which makes it hard to know when that's done, before jumping
to next task.

Test Plan

Render the interface, and you should see the next button provided.

image

To make labeling easier, provide the Next task button to allow labelers to iterate through items.

Test Plan:

Render the interface and should see the next button provided.
@dat-boris dat-boris self-assigned this Jul 25, 2024
@dat-boris dat-boris requested review from lizfaubell and a team July 25, 2024 21:43
Copy link

@lizfaubell lizfaubell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! One question about task list.

Comment on lines +764 to +766
// TODO: we should find better way to access the dataManager's Mobx Store
const taskList = window.dataManager.store.taskStore.list;
const thisTaskIdx = taskList.findIndex((x) => x.id === self.task.id);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this the task list for the specific tab we came from? Or all of them?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good question! Conveniently this is the list showed in the datamanager table, which is what is in the tab 🎉

I will add a comment on this!

@dat-boris dat-boris merged commit 6ae817e into master Jul 25, 2024
4 of 5 checks passed
khan-actions-bot pushed a commit to Khan/label-studio that referenced this pull request Jul 25, 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.

2 participants