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.

Sign up for GitHub

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

Jump to bottom

#Filtering movie tags and movie review mails #26

Merged
merged 2 commits into from
May 11, 2024
Merged

#Filtering movie tags and movie review mails #26

merged 2 commits into from
May 11, 2024

Conversation

yashwe0
Copy link
Contributor

@yashwe0 yashwe0 commented May 11, 2024 •
edited
Loading

Have made changes to the Movie_review_imdb_scrapping.ipynb to display the movie tags and movie review emails as desired :

  1. mentioned headers under getSoup(url) to handle 403 Error
  2. while filtering movie tags changed class : None to class : ipc-title-link-wrapper
  3. while filtering movie URLS made changes to movie_links
  4. mentioned n = 10 , 20 respectively in list[:n] where n values were to be displayed.

yashwe0 added 2 commits May 11, 2024 17:04
@yashwe0
Add files via upload
275621b
@yashwe0
Merge pull request #1 from yashwe0/yashwe0-patch-1
018b413 
#Filtering movie tags and Filtering movie review links
sanjay-kv
sanjay-kv approved these changes May 11, 2024
View reviewed changes
Copy link
Member

@sanjay-kv sanjay-kv left a comment

Choose a reason for hiding this comment

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

Good to go.

@sanjay-kv sanjay-kv merged commit afc4899 into recodehive:main May 11, 2024
1 check passed
@sanjay-kv
Copy link
Member

Please make sure you add it in the documentation as well so other people understand.

@yashwe0
Copy link
Contributor Author

yashwe0 commented May 11, 2024

I've edited my comment, is it okay for the documentation part?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanjay-kv sanjay-kv sanjay-kv approved these changes

Assignees

@yashwe0 yashwe0

Labels
gssoc level2
Projects
@sanjay-kv's Scrape ML Tracker
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yashwe0 @sanjay-kv