Skip to content

Fix Problem with .devcontainer. It did not copy .devcontainer/reinstall-cmake.sh when building container#2273

Open
GabrielReusRodriguez wants to merge 5 commits into
lemonade-sdk:mainfrom
GabrielReusRodriguez:dev_container_corrections
Open

Fix Problem with .devcontainer. It did not copy .devcontainer/reinstall-cmake.sh when building container#2273
GabrielReusRodriguez wants to merge 5 commits into
lemonade-sdk:mainfrom
GabrielReusRodriguez:dev_container_corrections

Conversation

@GabrielReusRodriguez

Copy link
Copy Markdown

Hello,

I saw the lemonade repo had a .devcontainer so I tryed the visual code devcontainer feature on my ubuntu 24.04 , visual code 1.124.2.

I tryed but i saw visual code throw an error when it tryed to build container

image

It failed when it tryed to copy .devcontainer/reinstall-cmake.sh . I investigate and i saw that inside .dockerignore there is a
.devcontainer line that is guilty :)

I added below :

!.devcontainer/reinstall-cmake.sh

so it will let to copy the script . I tryed and finally worked!

image

@github-actions github-actions Bot added bug Something isn't working area::ci CI / GitHub Actions / self-hosted runner infrastructure labels Jun 16, 2026

@fl0rianr fl0rianr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good fix, had the same issue every time but never pushed.

@GabrielReusRodriguez

Copy link
Copy Markdown
Author

Hello!

I have a question, i see That the tests failed on MAC whisper metal testing.

But I only changed the dockerignore , so I think it should not be affected.

Maybe is something related to: #2278 ?

Trace
......
Run set -e
Running whisper Metal inference tests...
test_001_transcription_basic (main.WhisperTests)
Test basic audio transcription with Whisper. ... skipped 'metal backend not available: {"error":{"code":"model_load_error","message":"Failed to load model 'Whisper-Tiny': whisper-server failed to start or become ready","param":"model","requested_model":"Whisper-Tiny","type":"model_load_error"}}'
test_002_transcription_with_language (main.WhisperTests)
Test audio transcription with explicit language parameter. ... FAIL

======================================================================
FAIL: test_002_transcription_with_language (main.WhisperTests)
Test audio transcription with explicit language parameter.

Traceback (most recent call last):
File "/Users/runner/work/lemonade/lemonade/test/server_whisper.py", line 182, in test_002_transcription_with_language
self.assertEqual(
AssertionError: 500 != 200 : Transcription failed with status 500: {"error":{"code":"model_load_error","message":"Failed to load model 'Whisper-Tiny': whisper-server failed to start or become ready","param":"model","requested_model":"Whisper-Tiny","type":"model_load_error"}}


Ran 2 tests in 5.088s

FAILED (failures=1, skipped=1)

======================================================================
WHISPER / AUDIO TRANSCRIPTION TESTS
Wrapped Server: whispercpp, Backend: metal

=== Verifying server is reachable ===
Server is reachable on port 13305
Applying runtime config: {'whispercpp': {'backend': 'metal'}}
Unloading all models for clean state...

[INFO] Downloading test audio file from https://raw.githubusercontent.com/lemonade-sdk/assets/main/audio/test_speech.wav
[OK] Downloaded to /var/folders/bh/cdl3cft92_dbvg852_fp7y7w0000gn/T/test_speech.wav

=== Starting test: test_001_transcription_basic ===
[INFO] Loading model with metal backend

=== Starting test: test_002_transcription_with_language ===
[INFO] Sending transcription request with language=en
[INFO] Cleaned up test audio file
Error: Process completed with exit code 1.

@fl0rianr fl0rianr enabled auto-merge June 17, 2026 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area::ci CI / GitHub Actions / self-hosted runner infrastructure bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants