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(image_list): sorting on scores works #164

Open
wants to merge 2 commits into
base: cat-filter-tweeks
Choose a base branch
from

Conversation

PaulHax
Copy link
Collaborator

@PaulHax PaulHax commented Jan 9, 2025

After updating the images visible in the image list, all other selected images are updated and their scores computed.
After images are scored, the table sort may have changed the images that are visible, so ImageList is asked to send visible image IDs again, which may trigger a new _update_all_images if the set of images in view has changed.

transformers.pipeline is not async, so we run that in subprocess via multiprocess_predictor.py

@PaulHax PaulHax linked an issue Jan 10, 2025 that may be closed by this pull request
@PaulHax PaulHax force-pushed the sort-fix branch 3 times, most recently from fc57b5a to eb7993b Compare January 10, 2025 18:03
@PaulHax PaulHax marked this pull request as ready for review January 10, 2025 18:24
@PaulHax PaulHax force-pushed the sort-fix branch 2 times, most recently from 5aad1f2 to 4149ea6 Compare January 10, 2025 18:52
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.

Fix score based sorting of sampled images in Image List
1 participant