Skip to content

Feat/text view class #479

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Feb 26, 2025
Merged

Feat/text view class #479

merged 4 commits into from
Feb 26, 2025

Conversation

@antoningaladrim
Copy link
Contributor

@antoningaladrim antoningaladrim commented Feb 25, 2025 •
edited
Loading

Description

  • Create TextView class and add the corresponding BaseSchema
  • Add two derived stores of views :
    • mediaViews: stores the Image and SequenceFrame arrays
    • textViews: stores the TextViews

Use mediaViews wherever views was used before. Leave the loadData method of datasetItemWorkspace as is.

To implement in future PR

  • Clean TextSpan removing annotationRef field
  • Refactor EntityLinking workspace with the updated/new types

@antoningaladrim antoningaladrim added scope: front Related to Pixano frontend enhancement New feature or request labels Feb 26, 2025
@juliendenize juliendenize merged commit 664e13e into main Feb 26, 2025
4 checks passed
@juliendenize juliendenize deleted the feat/text-view-class branch February 26, 2025 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@BertrandRenault BertrandRenault BertrandRenault approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

enhancement New feature or request scope: front Related to Pixano frontend

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@antoningaladrim @BertrandRenault @juliendenize