Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 8, 2025

Removes deprecated Keras-based ML training and inference code to be replaced by modern LLM trainers.

Deleted

  • train.py, generate.py - Keras training/generation scripts
  • loda/ml/keras/ - Complete RNN model implementation (335 lines)
  • TensorFlow dependencies from requirements.txt

Updated

  • sample.py - Removed Keras imports
  • tests/test_ml.py - Removed Keras imports (tests already disabled)
  • Documentation files - Removed ML/Keras references from README, copilot instructions

Preserved

  • loda/ml/util.py - Token conversion utilities still useful for future implementations

Net reduction: 458 lines removed. All 86 tests pass.

Original prompt

This section details on the original issue you should resolve

<issue_title>Remove Keras ML code</issue_title>
<issue_description>Remove the Keras ML code including all training and inference code. It is not needed anymore and will be replaced later by modern LLM trainers. Clean up the documentation and copilot instructions if needed.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: ckrause <840744+ckrause@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove Keras ML code and clean up documentation Remove Keras ML code Nov 8, 2025
Copilot AI requested a review from ckrause November 8, 2025 16:11
@ckrause ckrause marked this pull request as ready for review November 8, 2025 16:39
@ckrause ckrause requested a review from neoneye as a code owner November 8, 2025 16:39
@ckrause ckrause merged commit ed4cef3 into main Nov 8, 2025
2 checks passed
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.

Remove Keras ML code

2 participants