Skip to content

fix: make LLMModule and SpeechToTextModule non-static post C++ port#479

Merged
chmjkb merged 17 commits intomainfrom
@chmjkb/llm-stt-non-static
Jul 30, 2025
Merged

fix: make LLMModule and SpeechToTextModule non-static post C++ port#479
chmjkb merged 17 commits intomainfrom
@chmjkb/llm-stt-non-static

Conversation

@chmjkb
Copy link
Copy Markdown
Collaborator

@chmjkb chmjkb commented Jul 30, 2025

Description

Introduces a breaking change?

  • Yes
  • No

Type of change

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation update (improves or adds clarity to existing documentation)
  • Other (chores, tests, code style improvements etc.)

Tested on

  • iOS
  • Android

Testing instructions

Screenshots

Related issues

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@chmjkb chmjkb linked an issue Jul 30, 2025 that may be closed by this pull request
@chmjkb chmjkb marked this pull request as ready for review July 30, 2025 10:47
@chmjkb chmjkb requested a review from mkopcins July 30, 2025 10:48
Copy link
Copy Markdown
Member

@msluszniak msluszniak left a comment

Choose a reason for hiding this comment

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

Just some minor comments to the documentation.

Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/SpeechToTextModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/SpeechToTextModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/SpeechToTextModule.md Outdated
chmjkb and others added 8 commits July 30, 2025 13:06
…Module.md

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
…Module.md

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
…Module.md

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
…Module.md

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
…Module.md

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
…echToTextModule.md

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
…echToTextModule.md

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
…echToTextModule.md

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
Comment thread docs/docs/03-typescript-api/01-natural-language-processing/LLMModule.md Outdated
chmjkb and others added 5 commits July 30, 2025 14:07
Co-authored-by: Mateusz Kopcinski <120639731+mkopcins@users.noreply.github.com>
Co-authored-by: Mateusz Kopcinski <120639731+mkopcins@users.noreply.github.com>
…Module.md

Co-authored-by: Mateusz Kopcinski <120639731+mkopcins@users.noreply.github.com>
@chmjkb chmjkb requested a review from mkopcins July 30, 2025 12:21
@chmjkb chmjkb merged commit 536e1c4 into main Jul 30, 2025
3 checks passed
@chmjkb chmjkb deleted the @chmjkb/llm-stt-non-static branch July 30, 2025 12:33
mkopcins added a commit that referenced this pull request Oct 15, 2025
…479)

## Description

<!-- Provide a concise and descriptive summary of the changes
implemented in this PR. -->

### Introduces a breaking change?

- [x] Yes
- [ ] No

### Type of change

- [ ] Bug fix (change which fixes an issue)
- [x] New feature (change which adds functionality)
- [x] Documentation update (improves or adds clarity to existing
documentation)
- [ ] Other (chores, tests, code style improvements etc.)

### Tested on

- [ ] iOS
- [ ] Android

### Testing instructions

<!-- Provide step-by-step instructions on how to test your changes.
Include setup details if necessary. -->

### Screenshots

<!-- Add screenshots here, if applicable -->

### Related issues

<!-- Link related issues here using #issue-number -->

### Checklist

- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [ ] My changes generate no new warnings

### Additional notes

<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->

---------

Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
Co-authored-by: Mateusz Kopcinski <120639731+mkopcins@users.noreply.github.com>
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.

Make LLM & STT Modules non-static post C++ port

3 participants