keflavich / FITS_tools

Simple scipy and python based tools for FITS image manipulation
17 stars 12 forks source link

python setup.py build / test fails #1

Closed cdeil closed 7 years ago

cdeil commented 10 years ago

I get errors when I try to run python setup.py test or python setup.py build: https://gist.github.com/cdeil/121d7d90003d627f0613

@keflavich Do you want to make this an affiliated Astropy package and maintain it, or would it maybe make sense to combine this with python-reprojection?

cc @astrofrog

keflavich commented 10 years ago

I'm fine with either approach, though I'd prefer to find someone else to step in as maintainer (or at least, get some contributors).

I'm checking out the bug...

keflavich commented 10 years ago

@cdeil are you using py3?

keflavich commented 10 years ago

Nevermind - I couldn't reproduce this at first, but cleaning out conftest.py led to this error. As far as I can tell, it comes from astropy-helpers.

cdeil commented 10 years ago

Concerning FITS_tools and python-reprojection ... my main point is that both already have nice functionality, but are not ready for end-users due to lack of time of the few people working on them, so it could make sense to combine them to have reduced maintenance effort. But @astrofrog will probably know best what makes sense.

astrofrog commented 10 years ago

I've started to try and put reprojection stuff in one place: https://github.com/astropy/astropy/pull/2731

For now it just does the interpolation, but the idea would be to extend it to have the Montage version (I have a local branch that does that) and also the drizzle algorithm (the STScI devs will help with that).

astrofrog commented 10 years ago

This was done at the SciPy sprint and wasn't advertised yet but since you bring it up, just thought I'd mention it :) After discussions at the sprint, we decided it would make sense for this to live in the core package, not just an affiliated package, since reprojection is so fundamental to most observational astronomers.

keflavich commented 10 years ago

I can't figure out the bug. If I run python setup.py test a second time, it goes away, but

usage: -c [options] [file_or_dir] [file_or_dir] [...]
-c: error: unrecognized arguments: --doctest-rst

shows up

cdeil commented 10 years ago

I think this package has not been updated to use astropy-helpers? It probably doesn't make sense to debug such issues before that's done?

keflavich commented 10 years ago

I tried doing that earlier using subtree (second answer here: http://stackoverflow.com/questions/1425892/how-do-you-merge-two-git-repositories, not the full subtree package), so I may have done it wrong, but it claimed to be up to date.

cdeil commented 10 years ago

But the FITS_tools/setup.py in master has been last updated in May and is very different from the one in package-template!?

keflavich commented 10 years ago

yeah, I guess the merge just failed