Skip to content
This repository was archived by the owner on Dec 21, 2025. It is now read-only.

basic support for pubmedcentral #135

Merged
reece merged 3 commits into from
Aug 8, 2016
Merged

basic support for pubmedcentral #135

reece merged 3 commits into from
Aug 8, 2016

Conversation

@cariaso
Copy link
Contributor

@cariaso cariaso commented Aug 2, 2016 •
edited
Loading 

article = fetch.article_by_pmid('22416237')
pmcs = ec.efetch(db='pmc', id=article.pmc)
for pmc in pmcs:
    print pmc
    print pmc.title
    print pmc.abstract_text
    print pmc.body_text

@reece
Copy link
Member

reece commented Aug 4, 2016

Thanks for the PR, Mike!

Would you please address the test failures? (Example)
py.test README.rst eutils tests runs the tests in the current environment, and tox will run them in both 2.7 and 3.5.

@cariaso
Copy link
Contributor Author

cariaso commented Aug 4, 2016

yeah, happy to. When I first made the pull, I could see that the test was
failing, but was unable to get travis to show me the actual log to see
exactly what was failing. Your example works for me, so I can now address
that.

I'm still experimenting a bit with the exact api, and this isn't yet carved
in stone. suggestions or edits are welcome.

On Fri, Aug 5, 2016 at 12:54 AM, Reece Hart notifications@github.com
wrote:

Thanks for the PR, Mike!

Would you please address the test failures? (Example
https://travis-ci.org/biocommons/eutils/jobs/149720867)
py.test README.rst eutils tests runs the tests in the current
environment, and tox will run them in both 2.7 and 3.5.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#135 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHpkqLGtopke6cP4NT3u1n07qQULHciks5qcgsrgaJpZM4Jafvq
.

Mike Cariaso
http://www.cariaso.com

@reece reece merged commit 399b330 into biocommons:master Aug 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cariaso @reece