search

certik / mlc

Machine Learning Compiler
MIT License
9 stars 1 forks source link

start on gguf reader #32

Closed rebcabin closed 6 months ago

rebcabin commented 6 months ago

yes, there should not have been any changes to inference.py. This PR is supposed to have only new C code for displaying digits.

rebcabin commented 6 months ago

commit b45940edf8204dfe1f7cd83f3d6465ef7c85c886 (HEAD -> main, origin/main, origin/HEAD) Author: rebcabin rebcabin@users.noreply.github.com Date: Wed Mar 13 05:01:15 2024 -0700

fix incorrect changes to inference.py
rebcabin commented 6 months ago

This was an interesting case of Murphy's law. I was working on C code, copy-pasting from Python code. I was not paying attention to the Python code because I did not intend to change it. An inadvertent keystroke caused the selected Python code to be deleted, but I didn't notice. When I pushed the C code, the incorrectly changed Python code was pushed with it. Should be fixed now.

rebcabin commented 6 months ago

NOTE TO SELF: remember to set read-only-mode on files you don't intend to change.

certik commented 6 months ago

This is good enough to get started, let me merge it.