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

Providing Personalized Recommendations on the basis of results #20 #54

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ughrima
Copy link

@ughrima ughrima commented May 31, 2024

Screenshot from 2024-06-01 02-41-24

I have added personalised message for each detail. All of these messages will change for each particular input according to what is appropriate.
for eg - if Blood Pressure input is > 130 the high blood pressure message will be there, So on for each input

If diabetes is detected they will be given an option to know more about the disease, and if not detected then they can know how to prevent it - all with the blue button displayed at the bottom.

Copy link

vercel bot commented May 31, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
diabetes-prediction ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 7, 2024 11:14pm

@BamaCharanChhandogi
Copy link
Owner

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@ughrima
Copy link
Author

ughrima commented Jun 3, 2024

Hi @BamaCharanChhandogi Please revert to my PR

@BamaCharanChhandogi
Copy link
Owner

Hi @BamaCharanChhandogi Please revert to my PR

What happens?

@ughrima
Copy link
Author

ughrima commented Jun 4, 2024

What happens?

I just want to know if it will be merged or not.

@BamaCharanChhandogi
Copy link
Owner

BamaCharanChhandogi commented Jun 7, 2024

I have reviewed it. please make the changes. After changes, I could merge.

just like this-->

image

@ughrima
Copy link
Author

ughrima commented Jun 7, 2024

@BamaCharanChhandogi made the required changes, Kindly check

@BamaCharanChhandogi
Copy link
Owner

I was a little bit busy. Now we can work on our project.
you have made a lot of unnecessary changes.

  1. you do not have to show this log file. you can add it in the git-ignore file.
    image

  2. I think here you have to allow all origins.
    image

  3. Here, why did you add the logging command?
    image

@ughrima
Copy link
Author

ughrima commented Jun 26, 2024

I was a little bit busy. Now we can work on our project. you have made a lot of unnecessary changes.

  1. you do not have to show this log file. you can add it in the git-ignore file.
    image
  2. I think here you have to allow all origins.
    image
  3. Here, why did you add the logging command?
    image

I will fix this and push again

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.

2 participants