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

requirements.txt #15

Merged
merged 4 commits into from
Jul 15, 2016
Merged

Conversation

drphilmarshall
Copy link
Contributor

OK, I added a requirements.txt file for binder to pick up. The build of my fork is here - I'll keep committing changes to the requirements file until I can work through the whole thing with no errors. When you merge (not yet, please!), you'll need to rebuild your version of the tutorial by going to its binder status page and clicking "reload".

@fjaviersanchez
Copy link
Owner

I didn't want to include healpy in the requirements because it requires to have a C compiler installed. It is optional (the function where I use healpy can be executed as well without having it installed by setting healpy=False)

@drphilmarshall
Copy link
Contributor Author

Ah, OK - I'll remove this then. Thanks!

@drphilmarshall
Copy link
Contributor Author

I think you can merge this now - not sure why it is complaining about conflicts, I don't see any! Should be easy to fix in a command line merge. I issued #16 and #17 separately, because they are (probably) not binder problems. Thanks!

@fjaviersanchez fjaviersanchez merged commit 7f21ddb into fjaviersanchez:master Jul 15, 2016
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

Successfully merging this pull request may close these issues.

2 participants