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

Deployed the ml model seperately #196

Merged
merged 4 commits into from
Jan 28, 2025

Conversation

BhoomiAgrawal12
Copy link
Contributor

Fixes Issue🛠️

Closes #169

Description👨‍💻

I have deployed the model seperately on onrender and then made the api call in frontend,As checked in local setup the model is working fine and deployed properly.There is another app.py file being created under models and made the setup accordingly.
the hosted url is already being added to the frontend file,however if an env variable will be used it would be better so that future contributors can run it locally as well.

Type of Change📄

  • [ ✅] Bug fix (non-breaking change which fixes a bug)

Checklist✅

  • [ ✅] I am an Open Source contributor
  • [ ✅] I have performed a self-review of my code
  • [ ✅] My code follows the style guidelines of this project
  • [✅] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

Screenshot 2025-01-29 000315

Copy link

vercel bot commented Jan 28, 2025

@BhoomiAgrawal12 is attempting to deploy a commit to the Pratik0112's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @BhoomiAgrawal12, Welcome to 💖TelMedSphere !!! 🎊
Thanks for raising a PR! Your effort makes this project better. 🙌
Please wait for the PR to be reviewed.
Happy Coding!! ✨

@PratikMane0112
Copy link
Owner

@BhoomiAgrawal12 Thank you so much !!!!

@PratikMane0112 PratikMane0112 merged commit 4b776f3 into PratikMane0112:master Jan 28, 2025
1 of 3 checks passed
Copy link

@BhoomiAgrawal12 Congrats, Your pull request has been successfully merged 🥳🎉
Thank you for your contribution to 💖TelMedSphere !!!
Happy coding 🎊, Keep Contributing 🙌 !!!

@PratikMane0112
Copy link
Owner

@BhoomiAgrawal12, thank you for efforts

  • Can you update the UI as per the model requirement

  • Also create a new README.md for guide about the models, its features and deployment steps for new contributors

  • Also update the REAME.md to run models folder separately on local

  • Bhoomi, you can do this work in small small chunks & create a separate PR for each, So that your work will get good consideration & points

  • @AdityaBavadekar & @RajKhanke Also give your suggestion for it .

  • @BhoomiAgrawal12 Also you can tell issues/ observation or your suggestions w.r.t model for project

@BhoomiAgrawal12
Copy link
Contributor Author

Sure @PratikMane0112 , I will work on the next steps and also with time open new issues or changes with respect to the model.

@RajKhanke
Copy link
Collaborator

RajKhanke commented Jan 29, 2025

Model is working fine with multiple diseases , description and associated probabilities
image

@PratikMane0112
Copy link
Owner

PratikMane0112 commented Jan 29, 2025

Model is working fine with multiple diseases , description and associated probabilities image

Yes, I see. @BhoomiAgrawal12 is working on it. Thanks for adding up & your suggestions @RajKhanke

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

Successfully merging this pull request may close these issues.

[Bug] Large size of ML model causing memory issue on the deployment server & further process
3 participants