Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐕 Batch: Maintain additional credentials' requirements within models #1147

Open
DhanshreeA opened this issue Jun 5, 2024 · 4 comments
Open
Labels
discussion Ideas needed from the community enhancement New feature or request on-hold Interesting issue that we deprioritize

Comments

@DhanshreeA
Copy link
Member

Summary

Some models such as Fasmifra (eos4qda) benefit from additional credentials - eg eos4qda requires sudo credentials when run from source during fetch time. We anticipate the nature and scope of these requirements to expand in the future, for example we may need API keys from external vendors for some models and we need to maintain this information in the form of metadata.

Objective(s)

No response

Documentation

No response

@DhanshreeA DhanshreeA added documentation Improvements or additions to documentation enhancement New feature or request discussion Ideas needed from the community labels Jun 5, 2024
@DhanshreeA
Copy link
Member Author

Related to ersilia-os/eos4qda#2 (comment)

@GemmaTuron
Copy link
Member

Hi @DhanshreeA

Can you summarise here if there have been more discussions and plan around this?

@DhanshreeA
Copy link
Member Author

There has been a little development around this and @miquelduranfrigola started work on having environment variables for storing API keys for example for certain models, proposing to add dotenv to ersilia as a dependency. However we need to revisit this.

@miquelduranfrigola
Copy link
Member

Yes indeed - we need to make the necessary planning to push the authentication functionality forward. Not an immediate need but an important feature as we start including more and more remote API services such as ultra-large scale search or even LLMs.

As for dotenv, I am tempted to suggest that we implement this functionality in pure-python ourselves.

@GemmaTuron GemmaTuron removed the documentation Improvements or additions to documentation label Sep 24, 2024
@DhanshreeA DhanshreeA added the on-hold Interesting issue that we deprioritize label Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Ideas needed from the community enhancement New feature or request on-hold Interesting issue that we deprioritize
Projects
Status: On Hold
Development

No branches or pull requests

3 participants