Closed ramizhajj1 closed 4 months ago
I re-encoded the video.mp4 files using this command: ffmpeg -y -i "input.mp4" -c:v libx264 -pix_fmt yuv420p -preset superfast -crf 23 "output.mp4"
and it seems that the videos are correctly loading when I double click them on the unlinked videos table.
https://github.com/talmolab/sleap/assets/122569822/62f20429-cab0-4621-a070-7f555a80e194
[!WARNING]
Rate Limit Exceeded
@roomrys has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 6 minutes and 9 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 561df753bb4562406abbc7f490e8c81a61819960 and 05889f9d937463b45487347715f2e678de967eae.
The recent updates encompass various enhancements and modifications across multiple files in the SLEAP GUI application. These changes include improvements to video handling and GUI updates, addition of new encoding functions, and minor tweaks like adding print statements and blank lines. The modifications aim to improve functionality, maintainability, and user experience.
File(s) | Change Summary |
---|---|
Anaconda3-2023.04-MacOSX-x86_64.sh |
Returns a 404 Not Found error message with details about the missing key. |
sleap/gui/app.py |
Added methods for video handling and GUI updates, initialized GUI components, and set environment variables. |
sleap/gui/commands.py |
Added an extra blank line at the end of the file. |
sleap/gui/dataviews.py |
Added a print statement in the activateSelected method to display activation details. |
sleap/gui/scripts/reencoder.py |
Introduced functions to encode videos using ffmpeg with specified parameters. |
sleap/gui/widgets/docks.py |
Added is_activatable=True attribute to unlinked_videos_table and updated double-click handling. |
sleap/gui/widgets/video.py |
Connected update_selection_state to "unlinked_video" signal and added a blank line in load_video . |
sequenceDiagram
participant User
participant GUI
participant VideoHandler
participant Encoder
User->>GUI: Opens video file
GUI->>VideoHandler: Load video
VideoHandler->>GUI: Update GUI with video
User->>Encoder: Encode video
Encoder->>User: Provide encoded video
In the code where pixels play,
Videos dance and signals sway.
GUI refreshed, with views anew,
Encoding magic, just for you.
A rabbit hops in lines so neat,
Making SLEAP a coder's treat.
🌟✨🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 66.66667%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 74.08%. Comparing base (
ddeb083
) to head (05889f9
).
Files | Patch % | Lines |
---|---|---|
sleap/gui/widgets/video.py | 66.66% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
Set
SessionsDock.unlinked_videos_table.is_activatable = True
so thatstate["unlinked_video"]
is updated to the selectedVideo
item in theSessionDock.unlinked_videos_table
. Also make the callback forstate[unlinked_video"]
just update thestate["video"]
essentially making these states have the same callbacks.Types of changes
Does this address any currently open issues?
[list open issues here]
Outside contributors checklist
Thank you for contributing to SLEAP!
:heart: