litstudy icon indicating copy to clipboard operation
litstudy copied to clipboard
verified publisher icon NLeSC

Fix load scopus csv authors affiliations

Open tleedepriest opened this issue 2 years ago • 3 comments

  • added conftest file that supports simple tests on all docs in doc set
  • made scopus_csv more consistent with scopus source (returning None as opposed to empty list and vice versa)
  • add support for authors with multiple affiliations
  • add support for current CSV file download and older version
  • add tests for both versions and break tests out into independent tests.

tleedepriest avatar Aug 29 '23 23:08 tleedepriest

Hi Travis. Thank you very much this contribution. Fixing the ability to load Scopus CSV files is a highly requested feature. I am happy to see that this PR is well documented and has thorough testing.

I have made some (small) comments on the code.

stijnh avatar Aug 31 '23 09:08 stijnh

Hi Travis. Thank you very much this contribution. Fixing the ability to load Scopus CSV files is a highly requested feature. I am happy to see that this PR is well documented and has thorough testing.

I have made some (small) comments on the code.

Not sure how to look at the comments? Are you sure they are there or am I missing something?

tleedepriest avatar Sep 05 '23 12:09 tleedepriest

Hi.

You were right. They should be visible now!

stijnh avatar Sep 05 '23 13:09 stijnh