Skip to content

Conversation

@horatiualmasan
Copy link
Contributor

@horatiualmasan horatiualmasan commented Oct 21, 2025

What has changed and why?

Updated the Few shot classifier GUI elements. Create and refine processes are now dialogs above the blurred GUI.

How has it been tested?

Manual tests.

classifier_wizard.mp4

Did you update CHANGELOG.md?

  • Yes
  • Not needed (internal change)

@CLAassistant
Copy link

CLAassistant commented Oct 23, 2025

CLA assistant check
All committers have signed the CLA.

horatiualmasan and others added 3 commits October 23, 2025 17:27
…f github.com:lightly-ai/lightly-studio into horatiu-lig-7767-implement-new-fsc-createedit-wizard
@horatiualmasan horatiualmasan marked this pull request as ready for review October 23, 2025 14:40
…f github.com:lightly-ai/lightly-studio into horatiu-lig-7767-implement-new-fsc-createedit-wizard
@mihnea-necsulescu
Copy link
Contributor

after clicking on the refine classifier button until there are no more samples to refine, the wizard gets stuck like this:
image

I think this is because the samples infinite query will get disabled once the backend returns empty positive and negative sample ids, and $infiniteSamples.isPending will then be true indefinitely

we should probably update the ClassifierSamplesGrid to account for this too, and show a message saying that all samples have been used or something

Copy link
Contributor

@ikondrat ikondrat left a comment

Choose a reason for hiding this comment

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

lgtm

all my comments are optional and can be done in follow ups

@horatiualmasan horatiualmasan merged commit 823da92 into main Oct 24, 2025
6 checks passed
@horatiualmasan horatiualmasan deleted the horatiu-lig-7767-implement-new-fsc-createedit-wizard branch October 24, 2025 14:25
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.

5 participants