Skip to content

Issues 20260319#144

Merged
simoninns merged 3 commits intomainfrom
issues-20260319
Mar 21, 2026
Merged

Issues 20260319#144
simoninns merged 3 commits intomainfrom
issues-20260319

Conversation

@simoninns
Copy link
Copy Markdown
Owner

Summary

Fixes #138 issue with tape decoding where, in ld-decode-tools, ld-analyse contained a hack to preview a decoding path that ld-chroma-decoder did not support. Decode-Orc now implements this by improving the actual decoder (without requiring an ffmpeg processing loop) - and the orc-gui preview path is exactly the same as the decoder output path. So, it's not the same as the previous tool chain, but the results should be.

There was also a minor issue with parameters - the ffmpeg parameter for NTSC phase compensation was on by default, but off by default when the parameter editing window was open.

I added a bunch of unit testing to ffmpeg video sink, raw video sink and the underlying chroma decoder paths.

Issue #143 was around better support for agentic AI - now the project includes clear instructions for the agent to add additional colour to the existing documentation. I also updated the existing documentation to make everything clear.

Type of change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other

Validation

GUI testing and unit testing

Checklist

  • Scope is focused and minimal
  • Build passes locally
  • Related docs were updated if needed
  • Linked issue (if applicable)

Related issue

Closes #138 and #143

@simoninns simoninns merged commit 01a6f07 into main Mar 21, 2026
14 checks passed
@simoninns simoninns deleted the issues-20260319 branch March 21, 2026 14:30
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.

[Feature] Y/C Chroma Separation in Preview Window

1 participant