neokd / NeoGPT

Chat effortlessly, execute commands, and interpret code with Llama3, Phi3, and more - your local AI assistant. Enjoy seamless interaction while ensuring ultimate privacy
https://neogpt.dev
MIT License
78 stars 64 forks source link

Issue-111 #122

Closed bryce-seefieldt closed 9 months ago

bryce-seefieldt commented 9 months ago

Issue-111 - Partial implementation of config_export()

Proposed changes

Partial solution provided for Issue-111

Further discussion and clarification is required to

Checks

neokd commented 9 months ago

The function will be called in the CLI with --export xyz.yaml to export the current configuration to the file.

neokd commented 9 months ago

For number 2 we can just add the export date and time for now. Later on we can integrate other things in it.

neokd commented 9 months ago

@bryce-seefieldt any updates?

bryce-seefieldt commented 9 months ago

@neokd I think this Pull Request covers the creation of the export_config function. Please let me know if you have had a chance to review the code for the function and have any additions or changes you'd like to see implemented.

I have added the --export arg parse argument that would receive the export command line switch and run the export_config function. It saves the config settings in neogpt/settings directory. It allows user to provide an optional paramter for the config file name and defaults to settings.yml if no filename is provided.

neokd commented 9 months ago

@bryce-seefieldt the code looks good i have just reverted the UI config since there was a leak when using FAISS can you just make those small tweaks and create a new PR. also can you attach an config file?

neokd commented 9 months ago

@bryce-seefieldt Thanks for the contribution.