-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
feat: LEAP-1367: Add TimelineLabels to mark parts of the video #6191
Merged
Merged
Changes from 41 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
2200a7f
feat: LEAP-1362: Add TimelineLabels to mark parts of the video
hlomzik c64dfae
Add special appearance for timeline regions
hlomzik c042f70
Properly serialize timelinelabels
hlomzik d5689bc
Tiny fixes
hlomzik 549bb4f
ci: Build frontend
robot-ci-heartex fd7583a
Allow to draw only on empty space or special line
hlomzik e264642
Add quazi-line to create new regions
hlomzik d253ff3
Merge remote-tracking branch 'origin/fb-leap-1362/timeline-regions' i…
hlomzik e26a79f
Fix drawing on empty space (allowed) and region label (forbidden)
hlomzik 009832d
Show correct region index like in Outliner
hlomzik ffec5bf
Trigger CI
hlomzik 3fbd316
Merge branch 'develop' into 'fb-leap-1362/timeline-regions'
hlomzik 339beec
Add `timelineHeight` param to Video tag
hlomzik de8253b
Fix missing index passing + some optional types
hlomzik ed7744e
Add only the final state to undo history after drawing the region
hlomzik 7bdf8ed
Don't break existing Video rectangles regions
hlomzik 9abef00
Suppress timeline labels behaviour if there are no TimelineLabels
hlomzik f7422f0
Merge branch 'develop' into 'fb-leap-1362/timeline-regions'
hlomzik 490bf0c
Fix conditions for drawing frames in the timeline
hlomzik f368be7
Fix labels behaviour
hlomzik 7b879eb
Make TimelineLabels to behave like regular labels
hlomzik 730ee50
Disable extra controls for timeline regions
hlomzik d5f7cbc
Reverse regions only if TimelineLabels is presented
hlomzik 3369227
Fix and improve docs about TimelineLabels
hlomzik dae4848
Added a screenshot and a bit of text
f16a781
Fixing image link
bc245cc
Making fixes per Andrew's feedback
e9c9166
Fix RichText docs to unblock docs auto-generation
hlomzik e2599ce
ci: Build frontend
robot-ci-heartex 7777cc2
Update wording for results JSON
79529d7
Small comment about undo history management
hlomzik 4665000
Split adding rectangle and timeline regions
hlomzik 776480e
Merge branch 'develop' into fb-leap-1362/timeline-regions
hlomzik 2226699
ci: Build frontend
robot-ci-heartex bf32fb7
Fix issues with srubbing and hopping in Video
hlomzik 98c9c53
Merge branch 'develop' into 'fb-leap-1362/timeline-regions'
hlomzik 09110df
Tiny fixes, comments, renamed setSequence -> setRanges
hlomzik d827b31
Fix region restoration for VideoRectangles
hlomzik 4bb0985
Fix test to move indicator window back and forth
hlomzik 8dfcb2d
Fix hovered and selected state for all regions in video timeline
hlomzik ab969e8
Merge branch 'develop' into fb-leap-1362/timeline-regions
hlomzik 21f8ad5
Fix linting
hlomzik 56539fe
Style lint fixes
hlomzik f1272b6
ci: Build frontend
robot-ci-heartex File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,58 @@ | ||
--- | ||
title: TimelineLabels | ||
type: tags | ||
order: 429 | ||
is_new: t | ||
meta_title: TimelineLabels tag | ||
meta_description: Classify video frames using TimelineLabels. | ||
--- | ||
|
||
Use the TimelineLabels tag to classify video frames. This can be a single frame or a span of frames. | ||
|
||
First, select a label and then click once to annotate a single frame. Click and drag to annotate multiple frames. | ||
|
||
To move forward and backward in the timeline without labeling, ensure that no labels are selected before you click. | ||
|
||
![Screenshot of video with frame classification](../images/timelinelabels.png) | ||
|
||
Use with the `<Video>` control tag. | ||
|
||
!!! info Tip | ||
You can increase the height of the timeline using the `timelineHeight` parameter on the `<Video>` tag. | ||
|
||
### Parameters | ||
|
||
| Param | Type | Description | | ||
| --- | --- | --- | | ||
| name | <code>string</code> | Name of the element | | ||
| toName | <code>string</code> | Name of the video element | | ||
|
||
### Sample Results JSON | ||
|
||
| Name | Type | Description | | ||
| --- | --- | --- | | ||
| value | <code>Object</code> | | | ||
| value.ranges | <code>Array.<object></code> | Array of ranges, each range is an object with `start` and `end` properties. One range per region. | | ||
| [value.timelinelabels] | <code>Array.<string></code> | Regions are created by `TimelineLabels`, and the corresponding label is listed here. | | ||
|
||
### Example JSON | ||
```json | ||
{ | ||
"value": { | ||
"ranges": [{"start": 3, "end": 5}], | ||
"timelinelabels": ["Moving"] | ||
} | ||
} | ||
``` | ||
|
||
### Example | ||
```html | ||
<View> | ||
<Header>Label timeline spans:</Header> | ||
<Video name="video" value="$video" /> | ||
<TimelineLabels name="timelineLabels" toName="video"> | ||
<Label value="Nothing" background="#944BFF"/> | ||
<Label value="Movement" background="#98C84E"/> | ||
</TimelineLabels> | ||
</View> | ||
``` |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The
regions.map
call on this line is not used and can be removed to improve performance.