Skip to content

fixing aliyun asr#7

Merged
shenjinti merged 1 commit into
restsend:mainfrom
yeoleobun:aliyun-asr
Jul 18, 2025
Merged

fixing aliyun asr#7
shenjinti merged 1 commit into
restsend:mainfrom
yeoleobun:aliyun-asr

Conversation

@yeoleobun

Copy link
Copy Markdown
Collaborator

test passed

I have made some mistake on previous pr, after fixing it and force push, I cannot reopen it again. So here is another pr, sorry for that.

Comment thread src/transcription/aliyun.rs Outdated
pub punctuation: String,
}

const WS_URL: &str = "wss://dashscope.aliyuncs.com/api-ws/v1/inference";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

don't use const value

Comment thread src/transcription/aliyun.rs Outdated
continue;
}

assert!(response.header.event == "result-generated");

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

don't use assert!

@shenjinti
shenjinti merged commit 0aaad76 into restsend:main Jul 18, 2025
1 check failed
@yeoleobun
yeoleobun deleted the aliyun-asr branch August 5, 2025 03:05
yeoleobun pushed a commit to yeoleobun/rustpbx that referenced this pull request Mar 16, 2026
upgrade `actions/checkout@v4` to `actions/checkout@v6`
parvbhullar added a commit to parvbhullar/media-gateway that referenced this pull request Apr 16, 2026
…ests

- reload_populates_per_step_outcomes asserts steps.len()==3, names
  [trunks,routes,acl], and each step carries elapsed_ms + changed_count
- concurrent_reload_cas_conflict_returns_409 spawns two reload calls
  via tokio::spawn and asserts exactly one 200 + one 409
- Update reload_happy_path_shape to match the new 3-step shape
  (no 'app' step — deferred to Phase 11)
- Confirms SYS-02 truth restsend#7 + concurrent-race truth move to VERIFIED
shenjinti added a commit that referenced this pull request Jul 15, 2026
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