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

pubmed_scrapping #94

Merged
merged 1 commit into from
May 20, 2024
Merged

pubmed_scrapping #94

merged 1 commit into from
May 20, 2024

Conversation

VarsaGupta
Copy link
Contributor

Related Issue

GSSOC '24 : Scrapping extensive Pubmed data Using BioPython in few seconds. #48

Description

I added a folder scrapping_pubmed that contains pubmed.ipynb file that cotains the script of scrapping pubmed data both using Beautiful soup and BioPython. The comparison is show between them in the same file. Upon running the scripts respective json files are created which contains the data extracted from pubmed in json format.

Type of PR

  • Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): Added a pubmed_scrapping folder in the Scrape-ML directory.

Screenshots / videos (if applicable)

Here is the screenshot of the folder that I added in the Scrape-ML directory.
ss

Checklist:

  • I have performed a self-review of my code
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

Additional context:

Please feel free to provide any additional remarks, suggestions, or feedback concerning this pull request. Your input is valuable and appreciated.

@sanjay-kv sanjay-kv merged commit 3f199b5 into recodehive:main May 20, 2024
1 check passed
@VarsaGupta
Copy link
Contributor Author

Hey, @sanjay-kv, thankyou for merging my pull request. Could you please assign it a level ?

@sanjay-kv
Copy link
Member

@VarsaGupta sorry about that , i missed it. fixed it rn

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.

2 participants