makemore icon indicating copy to clipboard operation
makemore copied to clipboard

No updates to `self.logits` with bigram counts in Bigram class

Open mriganktiwari opened this issue 5 months ago • 1 comments

Hello everyone, i've made a makemore.py file (same as https://github.com/karpathy/makemore/blob/master/makemore.py) with only Bigram model class as of now.

while running it with python makemore.py -i names.txt -o names --type bigram arguments, wouldn't it just keep training, where it actually isn't doing anything coz there is no operation in forward pass.

Has anyone succesfully run it? Or it is yet to be completed?

mriganktiwari avatar Jul 16 '25 16:07 mriganktiwari

i have made changes that make it work, it's quite hacky as is. https://github.com/mriganktiwari/build_gpt/commit/e1140fda3fc3f5dac5563d86211b4e8486beb382

mriganktiwari avatar Jul 16 '25 18:07 mriganktiwari