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

BP Monitor Reading using DL #891

Closed
wants to merge 35 commits into from
Closed

Conversation

itspallavi20
Copy link

@itspallavi20 itspallavi20 commented Aug 9, 2024

Pull Request for DL-Simplified 💡

Issue Title : BP Monitor Reading using DL(modified)

  • Info about the related issue (Aim of the project) : Create a DL model, which will monitor the reading given in the images and identify them.
  • Name: Pallavi Patil
  • GitHub ID: https://github.com/itspallavi20
  • Email ID: [email protected]
  • Idenitfy yourself: (Mention in which program you are contributing in. Eg. For a JWOC 2022 participant it's, JWOC Participant) I am contributing in GSSOC(GirlScript Summer of Code)-2024

Closes: Initially the issue number was #245, after submitting previous PR which needed update, now it is #891.

Describe the add-ons or changes you've made 📃

(required corrections have been done ) I have created a DL model by implementing 3 algorithms-1.Custom CNN, 2.Resnet50, 3.RNN with CNN features and compared their accuracies and losses by training on medical dataset. Also updated by creating a single proper project folder consisting of subfolders containing Dataset, Images, Model-(DL model and README.md) and requirements.txt

Type of change ☑️

What sort of change have you made:

  • [x ] Bug fix (non-breaking change which fixes an issue)
  • [ x] New feature (non-breaking change which adds functionality)
  • [ x] Code style update (formatting, local variables)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [] This change requires a documentation update

How Has This Been Tested? ⚙️

The algorithms are compared on the basis of accuracy score and loss score. Graphs and visualizations are shown which reflect the scores for accuracy for each of the algorithm on the medical dataset for comparisons to find out the best algorithm for BP Monitor Reading dataset.

Checklist: ☑️

  • [ x] My code follows the guidelines of this project.
  • [x ] I have performed a self-review of my own code.
  • [ x] I have commented my code, particularly wherever it was hard to understand.
  • [x ] I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • [ x] I have added things that prove my fix is effective or that my feature works.
  • Any dependent changes have been merged and published in downstream modules.

Copy link

github-actions bot commented Aug 9, 2024

Our team will soon review your PR. Thanks @itspallavi20 :)

Copy link
Owner

@abhisheks008 abhisheks008 left a comment

Choose a reason for hiding this comment

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

image

Why you need so many project folders? Create a single project folder and add the sub folders accordingly. Remove the other project folders which are not required.

@abhisheks008 abhisheks008 added Status: Requested Changes Changes requested. gssoc Girlscript Summer of Code 2024 labels Aug 10, 2024
@itspallavi20
Copy link
Author

Required changes are updated in the single dedicated directory for BP Monitor Reading. Kindly review the changes, (PR #891)

@itspallavi20
Copy link
Author

itspallavi20 commented Aug 10, 2024 via email

@abhisheks008
Copy link
Owner

Please fix these typing mistakes, the heading tags are not working properly.
image

@itspallavi20

@itspallavi20
Copy link
Author

The corrections are fixed. Kindly review them.

@abhisheks008
Copy link
Owner

## Hello use this syntax only. Your markdown scripts are not working properly as the heading tags are not aligned.
@itspallavi20

@abhisheks008 abhisheks008 added wontfix ❌ This will not be worked on and removed Status: Requested Changes Changes requested. gssoc Girlscript Summer of Code 2024 labels Aug 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix ❌ This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants