Skip to content

What's new for 0.4 updates - #2694

Merged
eteq merged 5 commits into
astropy:masterfrom
eteq:whatsnew-updates
Jul 6, 2014
Merged

eteq merged 5 commits into
astropy:masterfrom
eteq:whatsnew-updates

Conversation

@eteq

@eteq eteq commented Jul 5, 2014

Copy link
Copy Markdown
Member

This PR adds to and polishes the what's new page, based more-or-less on some discussion from #2416 .

It adds a new Quantity section (@mhvk, you might want to glance over that), adds a variety of cleanup/missing stuff, and re-works the introduction to include some numbers about how many issues, PRs, and contributors were involved in this version.

For that last step, I used https://gist.github.com/eteq/9faf252ec8aa41188dcb, which uses the github api to search through all the issues and PRs closed since v0.3. That's worth noting for future versions, as it probably makes sense to reproduce this there.

cc @astrofrog @embray

@eteq eteq added docs labels Jul 5, 2014
@eteq eteq added this to the v0.4.0 milestone Jul 5, 2014
@eteq eteq mentioned this pull request Jul 5, 2014
1 of 2 tasks
Comment thread docs/whatsnew/0.4.rst Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@eteq - Quantity does not yet interact all that well with Table, so that should be removed. I think it would be good to state that "..., while work is continuing to integrate with ~astropy.table as well".

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

What I was thinking of was the most important use case to me - being able to do u.Quantity(table['somecol']) and having it include units if they are in the table (can't remember the PR number, but I really think that at least qualifies as "better" relative to v0.3). I'll re-phrase this to reflect that more is coming, though.

@mhvk

mhvk commented Jul 5, 2014

Copy link
Copy Markdown
Contributor

@eteq - apart from the comment on Quantity, note the duplicate use of "for details see ... for more details" in the coordinates section. Also, for cosmology, I'd use WMAP9.H(z=0.5), i.e., use an explicit keyword (assuming this is in fact correct... But the point is exactly that any example given here should ideally be completely understandable to even a novice.

@embray

embray commented Jul 5, 2014

Copy link
Copy Markdown
Member

Is this ready to go--can I merge it?

@eteq

eteq commented Jul 6, 2014

Copy link
Copy Markdown
Member Author

I implemented @mhvk's suggestions, so I think this is good, and will merge it now.

(@mhvk, if you don't like the change I made in Quantity/Table interaction, feel free to issue a separate PR - I'm merging now mostly so @embray can make any further what's new updates)

eteq added a commit that referenced this pull request Jul 6, 2014
@eteq
eteq merged commit a8797f8 into astropy:master Jul 6, 2014
@eteq
eteq deleted the whatsnew-updates branch July 6, 2014 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants