Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Dementia Prediction Model #212

Closed
wants to merge 4 commits into from

Conversation

SimranShaikh20
Copy link

@SimranShaikh20 SimranShaikh20 commented Nov 8, 2024

Description

Added new folder

Issue Resolved

issue no #208

Changes Made

Added folder including ,andatory files

Checklist

  • My code follows the current project structure
  • I have thoroughly reviewed and updated the requirements.txt file to include any new packages
  • The predict.py file includes a properly implemented model_details() function, or the notebook contains this function to print a detailed model report. The model will not be accepted without this function, as it is essential for generating the necessary model details.
  • I have added relevant tests (if necessary).
  • I have added comments in the code where needed.
  • This PR is submitted under Hacktoberfest.
  • This PR is submitted under GirlScript Summer of Code (GSSoC-Extd).

@yashasvini121 kindly review my PR if any conflicts then let me know !

Copy link
Owner

@yashasvini121 yashasvini121 left a comment

Choose a reason for hiding this comment

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

Although you've added 3 models, they currently lack the necessary project structure and importantly a form page.
Please refer to docs/tutorial and docs/project-structure for guidance. Feel free to reach out if you have any questions.

Copy link
Owner

Choose a reason for hiding this comment

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

Missing the model.py, predict.py, page for the model etc for the Dementia Prediction Model.
Please refer the docs/project-structure.md and docs/tutorial.md

Copy link
Owner

Choose a reason for hiding this comment

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

Follow the correct project structure and create a page for your model

@SimranShaikh20
Copy link
Author

@yashasvini121 so we have to also add streamlit app right?

@SimranShaikh20 SimranShaikh20 deleted the project3 branch November 8, 2024 18:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants