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

Create extractive_summarizer.py #26

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SparshRastogi
Copy link

This is the python code for an extractive text summariser that will use classical NLP techniques like TF-IDF to generate a summary for the given text
This is the Python file that was asked for before starting with the creation of proper tutorial
Closes issue

  • My project tutorial topic has already been aproved by the Codédex team
  • My project tutorial file renders properly in mdx
  • The content written is my own, and not plagiarized from other sources
  • All code written is commented or documented thoroughly
  • I have already completed a self-review of the project tutorial
  • I will comply with changes from the Codédex team about my project tutorial

This is the python code for an extractive text summariser that will use classical NLP techniques like TF-IDF to generate a summary for the given text
@SparshRastogi
Copy link
Author

@Bobliuuu this is the pull request providing py file as discussed for issue #24,can you please review it and suggest the required changes

Copy link
Contributor

@Bobliuuu Bobliuuu left a comment

Choose a reason for hiding this comment

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

Hi @SparshRastogi, very nice code! 🎉

Please just make sure to keep formatting consistent! This means spaces after commas but not before, either always or never having spaces between equal signs, and to do 0.2 * len(corpus) for better readability.

Once you make those changes, you can start the project tutorial! Please use the project template provided by Codedex for your project tutorial! You can check out past projects here for more info!

@SparshRastogi
Copy link
Author

@Bobliuuu thanks for the review,I will make the suggested changes and then start with the tutorial as per the template

@Bobliuuu
Copy link
Contributor

Hey @SparshRastogi, any updates on the project tutorial?

@SparshRastogi
Copy link
Author

@Bobliuuu sorry wasn't able to work upon it due to a hectic schedule past month,will try to complete it soon

@Bobliuuu
Copy link
Contributor

Hey @SparshRastogi, how's the project going?

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

Successfully merging this pull request may close these issues.

2 participants