search

containers / podman-desktop-extension-ai-lab

Work with LLMs on a local environment using containers
https://podman-desktop.io/extensions/ai-lab
Apache License 2.0
170 stars 31 forks source link

feat: instructLab navigation bar and empty InstructLab sessions screen #1615

Closed feloy closed 1 month ago

feloy commented 1 month ago

What does this PR do?

implement InstructLab navigation bar and empty InstructLab sessions screen

Screenshot / video of UI

ai-lab-tuning

What issues does this PR fix or reference?

Fixes #1567

How to test this PR?

You need to set an hidden setting, in ~/.local/share/containers/podman-desktop/configuration/settings.json

(you must completely stop Podman Desktop - not only close the window -, edit the config file, then restart Podman Desktop for the change to be taken into consideration) 

  "ai-lab.experimentalTuning": true,
jeffmaury commented 1 month ago

cc @jeffmaury @slemeur

Do we have some plan for the backend support and the implementation for interacting with Instructlab ? I am not sure to understand why we are starting to implement screens without having the logic to make it works.

The related epic #1517 seems to be mainly focused on design which is a bit odd ?

If the desired path is to have the screens before the logic, I would argue it should be hidden behind a flag, so it is not visible by default.

InstructLab is a WIP and backend issues will be created soon. But I agree (and I must confess) that all the InstructLab code should be conditionned by an environment variable

feloy commented 1 month ago

I have added an hidden setting: https://github.com/containers/podman-desktop-extension-ai-lab/pull/1615#issue-2491846391