Skip to content

Conversation

@Huge
Copy link

@Huge Huge commented Oct 31, 2023

I would have a look to integrate https://docs.python.org/3/library/getpass.html instead of( or parallel to) dotenv for accessing the LLM API here, that should be easier to get started fast.

@spirali
Copy link
Contributor

spirali commented Nov 1, 2023

Hi, thank you for your PR.

I think that it is generally good idea; however I am afraid of the situation that someone is running a development version (installed through poetry as we recommend it) and then open this notebook in the git repository and rewrites its own environment by the stable version from pip.

I have to think a little bit how to solve this situation.

For the getpass, I am totally ok with it is it is used when dotenv fails to load

@Huge
Copy link
Author

Huge commented Nov 3, 2023

Yeah, that is a valid concern. After a few tries I have found
!python -c "import dotenv,langchain,interlab" || pip install python-dotenv langchain interlab would work-around quite well, although it is not very elegant/legible and on a second look would better be one such line for each module.
I can update the PR code with this and getpass as fallback, just not sure if today.-)

Colab link in meanwhile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants