Skip to content

Commit 84a1df9

Browse files
authored
fix: fix case-sensitive filtering in LLMController special token loading (#666)
## Description <!-- Provide a concise and descriptive summary of the changes implemented in this PR. --> ### Introduces a breaking change? - [ ] Yes - [x] No ### Type of change - [x] 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 - [x] 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 - [x] 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 - [x] My changes generate no new warnings ### Additional notes <!-- Include any additional information, assumptions, or context that reviewers might need to understand this PR. -->
1 parent f9fcb04 commit 84a1df9

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

packages/react-native-executorch/src/controllers/LLMController.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -313,7 +313,7 @@ export class LLMController {
313313
const template = new Template(tokenizerConfig.chat_template);
314314

315315
const specialTokens = Object.fromEntries(
316-
Object.keys(SPECIAL_TOKENS)
316+
Object.values(SPECIAL_TOKENS)
317317
.filter((key) => key in tokenizerConfig)
318318
.map((key) => [key, tokenizerConfig[key]])
319319
);

0 commit comments

Comments
 (0)