Skip to content

Support variable-length frame indices (16-bit default, 32-bit overflow) - #2

Merged
Advik-B merged 9 commits into
masterfrom
copilot/implement-variable-length-frame-index
May 9, 2026
Merged

Support variable-length frame indices (16-bit default, 32-bit overflow)#2
Advik-B merged 9 commits into
masterfrom
copilot/implement-variable-length-frame-index

Conversation

Copilot AI commented May 9, 2026

Copy link
Copy Markdown
Contributor
  • Determine dynamic frame index sizing (16-bit default, 32-bit overflow) and update encoding metadata
  • Update decoder to use metadata-provided index width with 16-bit fallback
  • Refresh README capacity/limitations to reflect variable-length frame index

Copilot AI and others added 8 commits May 9, 2026 20:15
Agent-Logs-Url: https://github.com/Advik-B/FrameVault/sessions/635ce0d2-bc41-442a-bce7-05690a2115c1

Co-authored-by: Advik-B <86160411+Advik-B@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Advik-B/FrameVault/sessions/635ce0d2-bc41-442a-bce7-05690a2115c1

Co-authored-by: Advik-B <86160411+Advik-B@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Advik-B/FrameVault/sessions/635ce0d2-bc41-442a-bce7-05690a2115c1

Co-authored-by: Advik-B <86160411+Advik-B@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Advik-B/FrameVault/sessions/635ce0d2-bc41-442a-bce7-05690a2115c1

Co-authored-by: Advik-B <86160411+Advik-B@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Advik-B/FrameVault/sessions/635ce0d2-bc41-442a-bce7-05690a2115c1

Co-authored-by: Advik-B <86160411+Advik-B@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Advik-B/FrameVault/sessions/635ce0d2-bc41-442a-bce7-05690a2115c1

Co-authored-by: Advik-B <86160411+Advik-B@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Advik-B/FrameVault/sessions/635ce0d2-bc41-442a-bce7-05690a2115c1

Co-authored-by: Advik-B <86160411+Advik-B@users.noreply.github.com>
Agent-Logs-Url: https://github.com/Advik-B/FrameVault/sessions/635ce0d2-bc41-442a-bce7-05690a2115c1

Co-authored-by: Advik-B <86160411+Advik-B@users.noreply.github.com>
@Advik-B
Advik-B marked this pull request as ready for review May 9, 2026 20:22
@Advik-B

Advik-B commented May 9, 2026

Copy link
Copy Markdown
Owner

@copilot Run real tests against real files

Agent-Logs-Url: https://github.com/Advik-B/FrameVault/sessions/e3129a11-2efc-471c-8fc6-07ebd0bebf2e

Co-authored-by: Advik-B <86160411+Advik-B@users.noreply.github.com>

Copilot AI commented May 9, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Run real tests against real files

Ran an end-to-end encode/decode on a real 2KB binary file (ffmpeg + OpenCV installed) and confirmed SHA256 integrity PASS. Also fixed an indentation issue uncovered during the run in commit 238960f.

Copilot finished work on behalf of Advik-B May 9, 2026 20:39
Copilot AI requested a review from Advik-B May 9, 2026 20:39
Copilot finished work on behalf of Advik-B May 9, 2026 20:42
@Advik-B
Advik-B merged commit 10a6081 into master May 9, 2026
1 check passed
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.

2 participants