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

[FIX] passing session as non-positional argument #35

Merged

stijnh merged 1 commit into NLeSC:master from martinuray:master

Apr 5, 2023

Contributor

martinuray commented Apr 4, 2023

when executing refine_semanticscholar(...), the following error was thrown:

TypeError: fetch_semanticscholar() takes 1 positional argument but 2 were given

Fixed with this PR.


          [FIX] passing session as non-positional argument

d7d7d61

stijnh merged commit 72f0b60 into NLeSC:master

Member

stijnh commented Apr 5, 2023

Thanks for your contribution! This will be part of 1.0.6 release!

Not sure how this slipped through the tests, I'll look into this.

Peter9192 mentioned this pull request

refine_semanticscholar fails due to typeerror #41

Closed

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

Labels

None yet