Add a streamlit app for ersilia model

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

211 stars 147 forks source link

Add a streamlit app for ersilia model #126

Closed Amna-28 closed 2 years ago

Amna-28 commented 2 years ago

What is your feature request? Develop a streamlit app for ersilia model for better user experience

Is your feature request related to a problem? No, it could be just an enhancement

Describe alternatives you've considered with Ersilia model app, the user could select the model to run, choose api, fetch models, display catalog, request a cleanup (delete unused models in local machine) all in one place.

Additional context This will improve the usability of ersilia. I was researching and found that multiple tabs could be incorporated into a streamlit app, using which we can also improve the display of catalog ( #12) Links to the github repositories could also be added for each model in the catalog. There are many more possibilities.

screenshot of a simple app is attached below, that I developed while attempting the application tasks.

Capture2

Amna-28 commented 2 years ago

Hi @miquelduranfrigola , Please assign this task to me if you find it any useful. I would really enjoy working on it.

GemmaTuron commented 2 years ago

@Amna-28 great work.

Link this to your issue thread as well please. I will let @miquelduranfrigola comment. Something "nice to have" if the models work fine on streamlit is to show the representation of the molecular structure of the input (use the rdkit python library for example)

Amna-28 commented 2 years ago

Thanks @GemmaTuron , Sure thing, I will add the link and start working on it. Thanks for your feedback!

Amna-28 commented 2 years ago

Hi @GemmaTuron Keeping your suggestion in mind, are you talking about something like this?

Amna-28 commented 2 years ago

@GemmaTuron Can I consider this as a new idea mentioned in "other" section of the application as: Add a comment in this issue with your new idea, tagging the mentor

can I mark it referencing this issue?

miquelduranfrigola commented 2 years ago

Hi, @Amna-28 this is quite awesome!

A few suggestions:

Not all models have all APIs, so it would be great that the radio buttons in the "select API" are dynamic, depending on the available APIs for the model. If you are using the ErsiliaModel class, it has a get_apis method that can help.
It is OK to see the output in JSON format, but I think it is best if you just display the "output" part of it. No need for "input", "key", "text", etc.

What do you think?

GemmaTuron commented 2 years ago

I @Amna-28

Yes this is what I had in mind great job! This can be considered a new idea sure, no need to open an issue though since you are already solving it.

Amna-28 commented 2 years ago

@miquelduranfrigola Thank you for your suggestions, yes that would be better to just show the output part. Also I will make use of get_apis method to display api available in the model selected.

Amna-28 commented 2 years ago

I @Amna-28

Yes this is what I had in mind great job! This can be considered a new idea sure, no need to open an issue though since you are already solving it.

Got it, thank you @GemmaTuron for your continuous support and feedback