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

Add instructions on what are the prerequisite packages needed to run the code #7

Open
fedorov opened this issue Mar 22, 2017 · 11 comments
Assignees

Comments

@fedorov
Copy link
Collaborator

fedorov commented Mar 22, 2017

No description provided.

HesongHuang added a commit that referenced this issue Mar 28, 2017
#7 Add instructions on what are the prerequisite packages needed to run the code
@fedorov
Copy link
Collaborator Author

fedorov commented Apr 20, 2017

I will review and give feedback.

@fedorov
Copy link
Collaborator Author

fedorov commented Apr 26, 2017

I followed the instructions, and am using the latest version of the code from github.

When I run

python scholar.py -c 100 -T 6675397154864859782 --citation "bt"

I get this error:

scholar.py: error: no such option: -T

@HesongHuang
Copy link
Owner

Andrey,I am sure the error is from version of python. Could you download version 3?

@fedorov
Copy link
Collaborator Author

fedorov commented Apr 26, 2017

sure, I will try that

@fedorov
Copy link
Collaborator Author

fedorov commented Apr 28, 2017

I have problems installing matplotlib with python 3. I am not sure how to solve them quickly.

@HesongHuang
Copy link
Owner

If we decide to use JavaScript to make presentation. There is no need to use matplotlib. https://github.com/HesongHuang/scholar.py/blob/Javascript/Javascript.html
I will update the instruction and delete the matplotlib.

@fedorov
Copy link
Collaborator Author

fedorov commented Apr 28, 2017

I was able to run

python scholar.py/scholar.py -c 100 -T 6675397154864859782 --citation "bt"

However, I see the edits to instructions have already been integrated. The proper workflow is to first create a pull request with proposed changes, reference the issue it is supposed to resolve, and wait until you receive feedback before merging the pull request. This way any edits to the PR can be made before integration.

@fedorov
Copy link
Collaborator Author

fedorov commented Apr 28, 2017

Both journal_name.py and Pubmed_searchkeywords.py files depend on matplotlib or wordcloud, and I cannot install those packages for python 3.

@HesongHuang
Copy link
Owner

http://matplotlib.org/faq/installing_faq.html#install-osx-binaries
Does this instruction work?

@fedorov
Copy link
Collaborator Author

fedorov commented May 2, 2017

I don't have time to spend on this, sorry.

Please work on other issues. When do you plan to complete the courses I mentioned in #4?

@HesongHuang
Copy link
Owner

Now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants