Skip to content

Fix for testing Vulkan on Linux & Windows#40

Open
jacksongoode wants to merge 3 commits intocjpais:mainfrom
jacksongoode:gh-action-tests-2
Open

Fix for testing Vulkan on Linux & Windows#40
jacksongoode wants to merge 3 commits intocjpais:mainfrom
jacksongoode:gh-action-tests-2

Conversation

@jacksongoode
Copy link
Contributor

@jacksongoode jacksongoode commented Feb 24, 2026

A follow up to #39

Please have a look through this one, tests should pass but a few changes:

  • Using the tiny model (I'm sure we could use the medium, but I felt it wasn't really necessary) just for faster testing, though this exposed minor variations in transcription across platforms. Something that would need to be addressed anyway 🤷
  • Installing Vulkan with a somewhat up to date action
  • Using the GPU on macOS, but not Linux/Windows...

I'm not really sure about the GPU stuff, I don't think the Windows or Linux runners have GPUs?

After this it would be good to rebase #27 and #37 to get testing on their package bumps.

@jacksongoode jacksongoode changed the title Fix for Vulkan on Linux & Windows Fix for testing Vulkan on Linux & Windows Feb 24, 2026
@cjpais
Copy link
Owner

cjpais commented Mar 7, 2026

Im rewriting everything so sorry, I am ripping what was merged out, so this probably will need a rewrite too

@jacksongoode
Copy link
Contributor Author

No worries, we can fix the tests after.

@cjpais
Copy link
Owner

cjpais commented Mar 16, 2026

I think the right fix here is to disable Vulkan or be able to disable Vulkan so we can at least run CPU tests potentially

I'm probably gonna push this change in either today or tomorrow

@jacksongoode
Copy link
Contributor Author

I think the right fix here is to disable Vulkan

Okay that sounds fine, I can make the change or you, just make sure we're green before we merge :)

@cjpais
Copy link
Owner

cjpais commented Mar 17, 2026

Please feel free to :) I'm deep in 100 issues right now but would love to get testing working in ci here too

Tiny for whisper is surely fine :)

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