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

Text Summarization Model #147

Merged
merged 5 commits into from
Oct 19, 2024

Conversation

Peart-Guy
Copy link
Contributor

Description

I added a Text Summarizer Model
Name: Ankan
Github: Peart-Guy
Participant role: GSSOC-24 EXTD , Hacktoberfest

Issue Resolved

#139

Changes Made

I added a new model which summarizes long texts that enhances understanding and speed up learning—perfect for students and professionals alike.

Screenshots or Videos

text.summarizer.nlp.mp4

Additional Details

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).

@Peart-Guy
Copy link
Contributor Author

@yashasvini121 I have done my work. Kindly check it out and give GSSOC-24 EXTD, Hacktoberfest and Level 3. It will be an honor

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.

Your work in not in the correct project structure. Please refer docs/

@Peart-Guy
Copy link
Contributor Author

Your work in not in the correct project structure. Please refer docs/

Since my program is a NLP model so it is done in a single file and it does not need any separate code file to train or a separate code file to run or a separate file of data sets @yashasvini121

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.

Your final submission should include the following:

  1. An independent predict.py file using Streamlit.
  2. Your page added to the pages.json configuration.
  3. Your form configurations defined in form_config.
  4. Utilization of the page_handler class to generate your page.

Check docs in case, I've missed anything, but these are the primary requirements.

models/text_sumarization/predict.py Show resolved Hide resolved
@yashasvini121
Copy link
Owner

Feel free to reach out if you have any questions regarding the expected changes.

@Peart-Guy
Copy link
Contributor Author

Feel free to reach out if you have any questions regarding the expected changes.

@yashasvini121
Since my project comes with the built in UI so it is completed in one single file. My project does not need separate predict.py file or separate UI file. It will completely run by the Text Summarizer.py in the pages folder

I hope I was able to explain everything clearly. Would you like any further adjustments?

@yashasvini121
Copy link
Owner

Feel free to reach out if you have any questions regarding the expected changes.

@yashasvini121 Since my project comes with the built in UI so it is completed in one single file. My project does not need separate predict.py file or separate UI file. It will completely run by the Text Summarizer.py in the pages folder

I hope I was able to explain everything clearly. Would you like any further adjustments?

Your project works fine, but please note that the PR checklist specifies that the required project structure must be followed. If not, your PR will only receive a Level 1 rating.

@Peart-Guy
Copy link
Contributor Author

Feel free to reach out if you have any questions regarding the expected changes.

@yashasvini121 Since my project comes with the built in UI so it is completed in one single file. My project does not need separate predict.py file or separate UI file. It will completely run by the Text Summarizer.py in the pages folder
I hope I was able to explain everything clearly. Would you like any further adjustments?

Your project works fine, but please note that the PR checklist specifies that the required project structure must be followed. If not, your PR will only receive a Level 1 rating.

@yashasvini121
But I don't have extra files to add ,to maintain the project structure.
Kindly understand that my project is designed to be simple and efficient, requiring only a single file to function perfectly. This streamlined approach reflects my preference for minimalistic programming, ensuring both ease of maintenance and clarity in structure.
I trust that I have clearly explained the issue, and I am confident that you will understand my perspective.
Thank You

@Peart-Guy
Copy link
Contributor Author

Feel free to reach out if you have any questions regarding the expected changes.

@yashasvini121 Since my project comes with the built in UI so it is completed in one single file. My project does not need separate predict.py file or separate UI file. It will completely run by the Text Summarizer.py in the pages folder
I hope I was able to explain everything clearly. Would you like any further adjustments?

Your project works fine, but please note that the PR checklist specifies that the required project structure must be followed. If not, your PR will only receive a Level 1 rating.

@yashasvini121 But I don't have extra files to add ,to maintain the project structure. Kindly understand that my project is designed to be simple and efficient, requiring only a single file to function perfectly. This streamlined approach reflects my preference for minimalistic programming, ensuring both ease of maintenance and clarity in structure. I trust that I have clearly explained the issue, and I am confident that you will understand my perspective. Thank You

@yashasvini121 I hope you understood. Will be waiting for your reply
Thank You

@yashasvini121 yashasvini121 merged commit 246f52a into yashasvini121:master Oct 19, 2024
1 check passed
@yashasvini121 yashasvini121 linked an issue Oct 19, 2024 that may be closed by this pull request
@Peart-Guy
Copy link
Contributor Author

@yashasvini121 Thank You for Understanding. It was pleasure working with you.

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.

Text Summarizing NLP Model
2 participants