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

hopefully mpi4py is now optional for all of mock observables.... #20

Closed
wants to merge 7 commits into from
Closed

Conversation

duncandc
Copy link
Contributor

@duncandc duncandc commented Feb 5, 2015

No description provided.

@aphearin
Copy link
Contributor

aphearin commented Feb 5, 2015

Looks like Travis build is still failing. @eteq - I could be wrong, but the error appears to trace to a bug in _astropy_init.py.

Traceback (most recent call last):
File "", line 1, in
File "halotools/_astropy_init.py", line 111, in test
coverage=coverage, open_files=open_files, **kwargs)
TypeError: run_tests() got an unexpected keyword argument 'repeat'

@eteq
Copy link
Member

eteq commented Feb 5, 2015

@aphearin - it's actually a mix of different errors. Looking at https://travis-ci.org/astropy/halotools/builds/49669020 (the most recent one), it looks like the error you're noting is showing up for the last 3 tests, but not the others. I've created a new issue for that problem (#21).

If you look at, say 14.10 on that page, though, you'll see a different set of errors, all something to do with missing global variables. I think that's a real failure that @duncandc needs to correct, right?

@aphearin
Copy link
Contributor

aphearin commented Feb 6, 2015

Well, yes and no. The errors are definitely coming from the sector @duncandc works on, but it's still strange that when I run the test suite locally, there are no failures. I'm still not very practiced at deciphering Travis' traceback, but the errors seem to be stemming Duncan's attempts at parallel code. I don't know why this would be a problem on Travis and not on my machine.

@duncandc
Copy link
Contributor Author

duncandc commented Feb 6, 2015

Ok, I see at least one or two problems I can fix.

@duncandc duncandc closed this Feb 6, 2015
@eteq
Copy link
Member

eteq commented Feb 6, 2015

@aphearin - Travis is usually run on a vanilla version of ubuntu (12.something, I think?) in a virtualized environment, so there's all kinds of configuration things that might be different with regards to mpi. I'm confused why that would lead to this particular set of errors though... we'll see what @duncandc can come up with.

(Also, @duncandc , did you mean to close this?)

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.

None yet

3 participants