Skip to content

[WIP] Fix issues raised by coder.sbbitai#8

Merged
drQedwards merged 2 commits intodevelopfrom
copilot/fix-codersbbitai-issues
Feb 26, 2026
Merged

[WIP] Fix issues raised by coder.sbbitai#8
drQedwards merged 2 commits intodevelopfrom
copilot/fix-codersbbitai-issues

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 26, 2026

  • Fix flo_ai/flo_ai/tools/transformer_lattice_tool.py line 1: Remove invalid token prefix THIS SHOULD BE A LINTER ERROR that prevented Python from parsing the file
  • Fix transformer_lattice.c memory leak: Add destructor functions (free_head, free_mha, free_ffn, free_block, free_lattice) and call free_lattice(&net) in lattice_forward_api to prevent per-call memory leaks
  • Fix script.js duplicate sortProducts function: Renamed second sortProducts() (event handler) to handleSortChange() and updated index.html, fixing the function shadowing that caused infinite recursion when sorting
  • Run code review and security checks

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…icate function

1. Remove invalid 'THIS SHOULD BE A LINTER ERROR' prefix from
   transformer_lattice_tool.py that prevented Python parsing
2. Add destructor functions (free_head, free_mha, free_ffn, free_block,
   free_lattice) to transformer_lattice.c and call free_lattice(&net)
   in lattice_forward_api to fix per-call memory leaks
3. Rename duplicate sortProducts() event handler to handleSortChange()
   in script.js and index.html to fix function shadowing that caused
   infinite recursion

Co-authored-by: drQedwards <213266729+drQedwards@users.noreply.github.com>
@drQedwards drQedwards marked this pull request as ready for review February 26, 2026 18:43
@drQedwards drQedwards merged commit b93f466 into develop Feb 26, 2026
1 check passed
Copilot stopped work on behalf of drQedwards due to an error February 26, 2026 18:44
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