Skip to content

Conversation

@michal-lightly
Copy link
Contributor

@michal-lightly michal-lightly commented Oct 21, 2025

What has changed and why?

To be merged in a feature branch.

Used search and replace to rename:

  • class Sample* in lightly_studio/src/lightly_studio/models/sample.py to class Image*
    • Note that captions, tags and embeddings will stay linked to a sample - did not rename those
  • Renamed sample_table occurrences to image_table

Follow-up: Change the DB table name

How to review: Please check mostly the changes in models/ directory.

How has it been tested?

  • CI
  • Manually launched the app

Did you update CHANGELOG.md?

  • Yes
  • Not needed (internal change)

"""Sample class when inserting."""


class SampleViewForAnnotation(SQLModel):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was unused.

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

@michal-lightly michal-lightly merged commit 004ecbd into multimodal-db Oct 21, 2025
5 checks passed
@michal-lightly michal-lightly deleted the michal-lig-7853-rename-sampletable-to-imagetable-affects-fe-be--classes branch October 21, 2025 14:37
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.

3 participants