search

ericmjl / llamabot

Pythonic class-based interface to LLMs
https://ericmjl.github.io/llamabot/
124 stars 22 forks source link

Temperature shows N/A on log viewer when it's actually got a value #115

Open ericmjl opened 1 month ago

ericmjl commented 1 month ago
image

See screenshot! This looks like a very subtle bug in the UI code.

amirmolavi commented 1 month ago

Hi Eric, Is it okay if I look into this? How can I reproduce this? P.S. I started to look more in llamabot because I am trying to learn more about LLM and this is the way I learn, jump both feet in.

ericmjl commented 1 month ago

Hi Eric, Is it okay if I look into this?

Yes, please do so, @amirmolavi! I'd love to collaborate :smile:.

How can I reproduce this?

There should be a "getting started" guide on the README, which should boil down to the following commands: 

pixi install
pixi shell
llamabot visualize launch 
# this command name was given by an LLM and I really don't like it but it's stuck! 
# I think we should rename it to `llamabot logviewer launch` or something like that.

If the documentation feels sparse, please feel free to contribute as well -- I've had much success with drafting documentation for llamabot using llamabot docs write.

P.S. I started to look more in llamabot because I am trying to learn more about LLM and this is the way I learn, jump both feet in.

🎉 that's amazing! Thanks for choosing LlamaBot as the place to do your learning :smile:. As you get more familiar, let's see how we can build better cogs with LLMs!

amirmolavi commented 3 weeks ago

Hi Eric, I still have difficulties reproduce this. This is what I am getting now by running llamabot log-viewer launch:

image
ericmjl commented 3 weeks ago

@amirmolavi I just realized that some additional upgrades to the log-viewer UI broke the UI's ability to display logs. If you try now, does it work?

ericmjl commented 3 weeks ago

(I will admit I'm not the most seasoned at building UIs, so please bear with me as I muck around with it!)