Loading CSV file from Scopus about litstudy HOT 6 CLOSED

This has been partial resolved by #29. But it would be nice to have a load_scopus_csv function specifically for Scopus CSV files.

from litstudy.

I was just working with the exported Scopus CSV and was looking in the code base for this exact functionality. I would be happy to help! It would be my first open-source contribution.

from litstudy.

Hi Travis,

If you would like to contribute to litstudy by adding functionality for loading Scopus CSV files, we would be happy to review a pull request!

The steps for this would be roughly the following:

1. Fork this repository
2. Add a file litstudy/sources/scopus_csv.py
3. Add the function load_scopus_csv. You can base your work on ieee.py and springer.py
4. Add the load_scopus_csv function to litstudy/__init__.py and litstudy/sources/__init__.py
5. Add a test tests/test_sources_scopus_csv.py. You can base the test on test_sources_ieee.py
6. Format your code with the black formatting tool
7. Make a pull request for this repository. We can review your changes

Let me know if you run into any issues. Some more resources about creating pull requests:

• https://github.com/openpnp/openpnp/wiki/Your-First-Pull-Request
• https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request
• https://dev.to/doctolib/make-your-first-pull-request-to-an-open-source-project-1m57

If you want to give it a try, shall I assign this issue to you?

from litstudy.

Yes, I'd like to give it a go. I was currently working in scopus.py, but I will follow your suggestion and add a new file. Given that suggestion, would there be preference to create a new object ScopusCsvDocument, Instead of modifying the current ScopusDocument object to accommodate loading the csv?

from litstudy.

Great!

I don't think it is possible to use ScopusDocument since that class relies as pybliometrics and simply forwards all the method calls to pybliometrics. I feel like it would be difficult to modify this class to support CSV files. But if you find a way then that is also fine. It is probably best to add a new ScopusCsvDocument class

from litstudy.

This has now been resolved thanks to PR #45 (Thanks for your contribution tleedepriest!)

from litstudy.