Skip to content

fix assertion and linux build in rust test#392

Merged
HenryNdubuaku merged 1 commit intocactus-compute:mainfrom
yujonglee:attempt-to-fix-rust-ci
Feb 23, 2026
Merged

fix assertion and linux build in rust test#392
HenryNdubuaku merged 1 commit intocactus-compute:mainfrom
yujonglee:attempt-to-fix-rust-ci

Conversation

@yujonglee
Copy link
Copy Markdown
Contributor

@yujonglee yujonglee commented Feb 23, 2026

failing rust tests fixed

@yujonglee yujonglee force-pushed the attempt-to-fix-rust-ci branch 3 times, most recently from 2de4d6e to 72e3495 Compare February 23, 2026 14:53
…tability

- cactus_complete and cactus_transcribe return bytes written on success, not 0
- use u8 buffer with *mut c_char cast to fix i8/u8 mismatch on Linux ARM
- stt test now uses real jfk.wav audio and asserts non-empty transcription

Signed-off-by: Yujong Lee <[email protected]>
@yujonglee yujonglee force-pushed the attempt-to-fix-rust-ci branch from c6b63f4 to 929e136 Compare February 23, 2026 15:10
@HenryNdubuaku HenryNdubuaku merged commit a6caaf7 into cactus-compute:main Feb 23, 2026
3 of 4 checks passed
@yujonglee yujonglee deleted the attempt-to-fix-rust-ci branch February 23, 2026 22:59
ncylich pushed a commit that referenced this pull request Feb 24, 2026
…tability (#392)

- cactus_complete and cactus_transcribe return bytes written on success, not 0
- use u8 buffer with *mut c_char cast to fix i8/u8 mismatch on Linux ARM
- stt test now uses real jfk.wav audio and asserts non-empty transcription

Signed-off-by: Yujong Lee <[email protected]>
cattermelon1234 pushed a commit to cattermelon1234/cactus that referenced this pull request Feb 28, 2026
…tability (cactus-compute#392)

- cactus_complete and cactus_transcribe return bytes written on success, not 0
- use u8 buffer with *mut c_char cast to fix i8/u8 mismatch on Linux ARM
- stt test now uses real jfk.wav audio and asserts non-empty transcription

Signed-off-by: Yujong Lee <[email protected]>
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