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

More fixes to the LDO mask handling, etc. #394

Merged
merged 24 commits into from
Jun 3, 2017

Conversation

keflavich
Copy link
Contributor

This revisits issue #392, which wasn't actually solved by that PR. However, it
turns out we need to make deeper changes.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 75.353% when pulling dd3a86e on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 75.353% when pulling af0f972 on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 75.264% when pulling c616e22 on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 75.264% when pulling 893d591 on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 75.264% when pulling d16fccb on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@keflavich keflavich requested a review from e-koch June 2, 2017 03:22
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 75.264% when pulling c7d60ec on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@keflavich
Copy link
Contributor Author

I have a case where I'm getting an error that I can't come up with a test for:

In [3]: cube[:10,20,20].max()
Out[3]: Traceback (most recent call last):
  File "/Users/adam/anaconda/envs/astropy35/lib/python3.5/site-packages/IPython/core/formatters.py", line 693, in __call__
    printer.pretty(obj)
  File "/Users/adam/anaconda/envs/astropy35/lib/python3.5/site-packages/IPython/lib/pretty.py", line 380, in pretty
    return _default_pprint(obj, self, cycle)
  File "/Users/adam/anaconda/envs/astropy35/lib/python3.5/site-packages/IPython/lib/pretty.py", line 495, in _default_pprint
    _repr_pprint(obj, p, cycle)
  File "/Users/adam/anaconda/envs/astropy35/lib/python3.5/site-packages/IPython/lib/pretty.py", line 693, in _repr_pprint
    output = repr(obj)
  File "/Users/adam/repos/spectral-cube/spectral_cube/lower_dimensional_structures.py", line 481, in __repr__
    arrstr = np.array2string(self.filled_data[:].value, separator=',',
  File "/Users/adam/repos/spectral-cube/spectral_cube/cube_utils.py", line 203, in __getitem__
    return self._func(self._other, view)
  File "/Users/adam/repos/spectral-cube/spectral_cube/base_class.py", line 273, in filled_data
    return u.Quantity(self._get_filled_data(view, fill=self._fill_value),
  File "/Users/adam/repos/astropy/astropy/units/quantity.py", line 918, in __getattr__
    attr))
AttributeError: 'OneDSpectrum' object has no '_fill_value' member

@keflavich
Copy link
Contributor Author

How do we return a Quantity if a ufunc reducing to zero dimensions is run?

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 75.295% when pulling 6ebe24f on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 75.333% when pulling 359d565 on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 75.333% when pulling 359d565 on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@e-koch
Copy link
Contributor

e-koch commented Jun 3, 2017

@keflavich -- I reverted all of the changes for 2D masking.

Copy link
Contributor

@e-koch e-koch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

The other issues discussed here are with regards to 2D masking and WCS objects without both celestial axes defined.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 75.333% when pulling e6230e1 on keflavich:issue392_revisited into e2b9019 on radio-astro-tools:master.

@e-koch e-koch merged commit 4cb0512 into radio-astro-tools:master Jun 3, 2017
@keflavich keflavich deleted the issue392_revisited branch June 3, 2017 23:34
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.

3 participants