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

Marvin Deprecation Error. #6

Open
karenlmasters opened this issue Jul 15, 2020 · 3 comments
Open

Marvin Deprecation Error. #6

karenlmasters opened this issue Jul 15, 2020 · 3 comments

Comments

@karenlmasters
Copy link

I get the following Marvin Deprecation Error running the trial Notebook which calls this code. Not quite sure what changed with Marvin that breaks this.... can you help me figure it out?

MarvinDeprecationError                    Traceback (most recent call last)
<ipython-input-5-50dda1424bbf> in <module>
      4 data.get_bpt()
      5 data.make_all_spaxel_masks()
----> 6 data.get_mean_spectra(inv=True)

/raid5/homes/klmasters/Galaxy Zoo 3D/gz3d_fits.py in get_mean_spectra(self, inv)
    226         if self.cube != 'no_data':
    227             self.make_all_spaxel_masks()
--> 228             self.mean_bar = self._stack_spectra('bar_mask_spaxel')
    229             self.mean_spiral = self._stack_spectra('spiral_mask_spaxel')
    230             self.mean_center = self._stack_spectra('center_mask_spaxel')

/gz3d_fits.py in _stack_spectra(self, mask_name, inv)
    212         if len(mdx[0] > 0):
    213             weights = mask[mdx]
--> 214             spaxels = self.cube[mdx]
    215             spectra = np.array([s.spectrum for s in spaxels])
    216             if len(spectra) == 1:

marvin_subclass.py in __getitem__(self, xy)
    111     def __getitem__(self, xy):
    112         """Returns the spaxel for ``(x, y)`` without propserties"""
--> 113         spaxel = self.getSpaxel(x=xy[0], y=xy[1], properties=False, xyorig='lower')
    114         if isinstance(spaxel, list):
    115             for s in spaxel:

/packages/python3.6.8/lib/python3.6/site-packages/marvin/tools/cube.py in getSpaxel(self, x, y, ra, dec, maps, modelcube, **kwargs)
    576                 raise marvin.core.exceptions.MarvinDeprecationError(
    577                     'the {0} parameter has been deprecated. '
--> 578                     'Use maps or modelcube.'.format(old_param))
    579 
    580         return marvin.utils.general.general.getSpaxel(x=x, y=y, ra=ra, dec=dec,

MarvinDeprecationError: the properties parameter has been deprecated. Use maps or modelcube.
You can submit this error to Marvin GitHub Issues (https://github.com/sdss/marvin/issues/new).
Fill out a subject and some text describing the error that just occurred.
If able, copy and paste the full traceback information into the issue as well.```
@karenlmasters
Copy link
Author

That said. A lot of the code after this does work!

@CKrawczyk
Copy link
Owner

CKrawczyk commented Jul 16, 2020

Well, that is annoying. I guess there is a new way to access the datacube...

@karenlmasters
Copy link
Author

The Marvin error maybe on the Marvin side (due to the MPL-10 updates) so hang tight on that, unless you know its something which has really changed in Marvin!

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

No branches or pull requests

2 participants