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

Cone search test failure for 2.0rc1 on py2.7 #6297

Closed
stargaser opened this issue Jun 28, 2017 · 1 comment
Closed

Cone search test failure for 2.0rc1 on py2.7 #6297

stargaser opened this issue Jun 28, 2017 · 1 comment

Comments

@stargaser
Copy link
Contributor

Here is a failure in a python=2.7 conda environment on 2.0rc1:

1 failed, 12472 passed, 234 skipped, 69 xfailed, 1 xpassed, 43908 warnings in 454.52 seconds

=================================== FAILURES ===================================
_______________ TestConeSearchValidation.test_validation[False] ________________

self = <astropy.vo.validator.tests.test_validate.TestConeSearchValidation object at 0x1223e1d50>
parallel = False

    @pytest.mark.parametrize(('parallel'), [True, False])
    def test_validation(self, parallel):
        if os.path.exists(self.out_dir):
            shutil.rmtree(self.out_dir)

        validate.check_conesearch_sites(
            destdir=self.out_dir, parallel=parallel, url_list=None)

        for val in self.filenames.values():
            self._compare_catnames(get_pkg_data_filename(
                os.path.join(self.datadir, val)),
>               os.path.join(self.out_dir, val))

anaconda/envs/astropytstpy2/lib/python2.7/site-packages/astropy/vo/validator/tests/test_validate.py:77:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

fname1 = '/Users/shupe/anaconda/envs/astropytstpy2/lib/python2.7/site-packages/astropy/vo/validator/tests/data/conesearch_warn.json'
fname2 = '/var/folders/t6/zv_75s3156780r5wyknqyb8w0000gn/T/tmpDMvT4v/conesearch_warn.json'

    @staticmethod
    def _compare_catnames(fname1, fname2):
        db1 = VOSDatabase.from_json(fname1)
        db2 = VOSDatabase.from_json(fname2)
>       assert db1.list_catalogs() == db2.list_catalogs()
E       AssertionError: assert [] == ['The USNO-A2.0 Catalogue (Monet+ 1998) 1']
E         Right contains more items, first extra item: 'The USNO-A2.0 Catalogue (Monet+ 1998) 1'
E         Use -v to get the full diff

anaconda/envs/astropytstpy2/lib/python2.7/site-packages/astropy/vo/validator/tests/test_validate.py:64: AssertionError
----------------------------- Captured stdout call -----------------------------
INFO: good: 0 catalog(s) [astropy.vo.validator.validate]
INFO: warn: 1 catalog(s) [astropy.vo.validator.validate]
INFO: excp: 0 catalog(s) [astropy.vo.validator.validate]
INFO: nerr: 0 catalog(s) [astropy.vo.validator.validate]
INFO: total: 1 out of 1 catalog(s) [astropy.vo.validator.validate]
INFO: check_conesearch_sites took 0.698978185654 s on AVERAGE for 1 call(s). [astropy.utils.timer]

Test setup details (OS X 10.12.5):

python -c "import astropy; astropy.test(remote_data=True)"
============================= test session starts ==============================
platform darwin -- Python 2.7.13, pytest-3.1.2, py-1.4.34, pluggy-0.4.0

Running tests with Astropy version 2.0rc1.
Running tests in anaconda/envs/astropytstpy2/lib/python2.7/site-packages/astropy.

Date: 2017-06-28T09:17:07

Platform: Darwin-16.6.0-x86_64-i386-64bit

Executable: /Users/shupe/anaconda/envs/astropytstpy2/bin/python

Full Python Version:
2.7.13 |Continuum Analytics, Inc.| (default, Dec 20 2016, 23:05:08)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.57)]

encodings: sys: ascii, locale: UTF-8, filesystem: utf-8, unicode bits: 15
byteorder: little
float info: dig: 15, mant_dig: 15

Numpy: 1.13.0
Scipy: 0.19.0
Matplotlib: 2.0.2
h5py: 2.7.0
Pandas: not available
Cython: 0.25.2
Using Astropy options: remote_data: any.
@pllim
Copy link
Member

pllim commented Jun 28, 2017

As far as the RC is concerned, you can ignore this failure. Cone Search is deprecated in #5904 for v2 and its tests (including this one) will be removed in v3.

But if you actually uses Cone Search and are concerned, please open an issue at Astroquery.

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

No branches or pull requests

2 participants