Skip to content
This repository was archived by the owner on Aug 31, 2025. It is now read-only.

Conversation

@OlegJakushkin
Copy link

A headless Dockerfile + a small .sh file to coordinate audio length that renders video.

Tested to work (and produce video), yet sound seems not to be passed on correctly - may you could give me some tips on how to pass audio into it properly -please point me to something if there is a test one could run for audio working sanity check..?

Should also probably fix CI...

@OlegJakushkin OlegJakushkin marked this pull request as ready for review July 13, 2025 06:47
@OlegJakushkin
Copy link
Author

@polyfloyd, could you please point me to some tests of audio passing that should work?

@OlegJakushkin
Copy link
Author

Dear @polyfloyd ,
Got audio sync working, Updated Github WF testing compilation.
Rendered Example 200x200 of this shader

Also, no unit tests fail due to some code error, and not build one, please review:

Run go test -v -race -cover ./...
=== RUN   TestParseGeometry
=== RUN   TestParseGeometry/valid
=== RUN   TestParseGeometry/invalid
--- PASS: TestParseGeometry (0.00s)
    --- PASS: TestParseGeometry/valid (0.00s)
    --- PASS: TestParseGeometry/invalid (0.00s)
PASS
coverage: 5.4% of statements
ok  	github.com/polyfloyd/shady/cmd/shady	1.085s	coverage: 5.4% of statements
	github.com/polyfloyd/shady/egl		coverage: 0.0% of statements
	github.com/polyfloyd/shady/encode		coverage: 0.0% of statements
=== RUN   TestSingleFileMarkerPosition
    compile_test.go:19: 
        
    compile_test.go:23: Expected at least one error marker
--- FAIL: TestSingleFileMarkerPosition (2.19s)
=== RUN   TestMultiFileMarkerPosition
    compile_test.go:48: 
        1:3(2): preprocessor error: #error meh
        
--- PASS: TestMultiFileMarkerPosition (0.01s)
=== RUN   TestPlain
--- PASS: TestPlain (0.00s)
=== RUN   TestIncludeSingle
--- PASS: TestIncludeSingle (0.00s)
=== RUN   TestIncludeRecursive
--- PASS: TestIncludeRecursive (0.00s)
=== RUN   TestStopRecursionCycle
--- PASS: TestStopRecursionCycle (0.00s)
FAIL
coverage: 10.9% of statements
FAIL	github.com/polyfloyd/shady/renderer	2.215s
	github.com/polyfloyd/shady/shadertoy		coverage: 0.0% of statements
	github.com/polyfloyd/shady/shadertoy/audio		coverage: 0.0% of statements
	github.com/polyfloyd/shady/shadertoy/image		coverage: 0.0% of statements
	github.com/polyfloyd/shady/shadertoy/kinect		coverage: 0.0% of statements
	github.com/polyfloyd/shady/shadertoy/peripheral		coverage: 0.0% of statements
	github.com/polyfloyd/shady/shadertoy/video		coverage: 0.0% of statements
FAIL
Error: Process completed with exit code 1.

Mesa ≥ 23.2 tweaked the shader-compiler log format:

Example	Mesa ≤ 23.1	Mesa ≥ 23.2
single file, #error	0(3) : …	0:3: …
Fixes Dockerfile.test tests be able to pass
@OlegJakushkin
Copy link
Author

@polyfloyd now with mesa parsing fix tests shall be green.
Locally tested via docker build -f .\Dockerfile.test .
Note: changed line endings to Linux ones.

@polyfloyd
Copy link
Owner

Hey, thanks.

I am not too keen om upstreaming specific workflows like rendering videos with audio like this, but I would be happy to cherry-pick a few CI things when I have some time

@polyfloyd polyfloyd closed this Aug 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants