search

ersilia-os / ersilia

The Ersilia Model Hub, a repository of AI/ML models for infectious and neglected disease research.
https://ersilia.io
GNU General Public License v3.0
218 stars 147 forks source link

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

Open DhanshreeA opened 5 months ago

DhanshreeA commented 5 months ago

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 commented 5 months ago

Related to https://github.com/ersilia-os/eos4qda/issues/2#issuecomment-2144831029

GemmaTuron commented 1 month ago

Hi @DhanshreeA

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

DhanshreeA commented 1 month ago

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 commented 1 month ago

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.