Skip to content

Allow the spectral_density equivalency to take a Quantity - #1590

Closed
astrofrog wants to merge 1 commit into
astropy:masterfrom
astrofrog:spectral-density-argument
Closed

Allow the spectral_density equivalency to take a Quantity#1590
astrofrog wants to merge 1 commit into
astropy:masterfrom
astrofrog:spectral-density-argument

Conversation

@astrofrog

@astrofrog astrofrog commented Oct 16, 2013

Copy link
Copy Markdown
Member

@mdboom - what do you think? I had to use the current API today and it felt clunky (also, other equivalencies now take quantities). Should we actually deprecate the previous API?

@mdboom

mdboom commented Oct 16, 2013

Copy link
Copy Markdown
Contributor

Yeah -- the original API predates Quantity.

I think #1560 already does this (as part of some larger changes).

@astrofrog

astrofrog commented Oct 16, 2013

Copy link
Copy Markdown
Member Author

Ah yes, then I'll leave this open for now but it can be closed without merging once #1560 is merged.

@eteq

eteq commented Oct 16, 2013

Copy link
Copy Markdown
Member

FYI, I think this will be much less important when we have a Flux object (#1577 , although I'm not sure if it actually implements this feature), as you never have to bother with the actual equivalency if you use that. This was actually the main use case in my head for being able to store equivalencies in quantities.

@pllim

pllim commented Oct 17, 2013

Copy link
Copy Markdown
Member

Ops... sorry for the duplicate work. I like the backward compatibility option here, which is not in my PR.

@astrofrog

astrofrog commented Oct 17, 2013

Copy link
Copy Markdown
Member Author

@pllim - feel free to implement that in your PR and we can just close this one once yours is merged :)

pllim added a commit to pllim/astropy that referenced this pull request Oct 17, 2013
@pllim
Added text to CHANGES.rst as suggested by @eteq. Applied changes from a…
b3dfdeb 
…stropy#1590 to this PR, as suggested by @astrofrog.
@pllim pllim mentioned this pull request Oct 17, 2013
Closed
eteq pushed a commit to eteq/astropy that referenced this pull request Oct 17, 2013
@pllim @eteq
Added text to CHANGES.rst as suggested by @eteq. Applied changes from a…
935350e 
…stropy#1590 to this PR, as suggested by @astrofrog.
@pllim

pllim commented Oct 18, 2013

Copy link
Copy Markdown
Member

@eteq -- This should be closed too. It was implemented in #1560 that you just merged. FYI.

@eteq

eteq commented Oct 18, 2013

Copy link
Copy Markdown
Member

Good call, thanks @pllim

@eteq eteq closed this Oct 18, 2013
@astrofrog
astrofrog deleted the spectral-density-argument branch July 5, 2016 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v0.3.0

Development

Successfully merging this pull request may close these issues.

4 participants

@astrofrog @mdboom @eteq @pllim