Skip to content

Manual backport of #11145 (Use proper precision when format not specified) - #11416

Merged
eteq merged 2 commits into
astropy:v4.2.xfrom
pllim:backport-pr11145
Mar 25, 2021
Merged

eteq merged 2 commits into
astropy:v4.2.xfrom
pllim:backport-pr11145

Conversation

@pllim

@pllim pllim commented Mar 24, 2021 •

Copy link
Copy Markdown
Member

Use proper precision when format not specified

Description

This pull request is to manually backport #11145 for 4.2.1.

…format

Use proper precision when format not specified
@pllim pllim added this to the v4.2.1 milestone Mar 24, 2021
Comment thread CHANGES.rst
- Ensure ``keepdims`` works for taking ``mean``, ``std``, and ``var`` of
``Quantity``. [#11198]

- For ``Quantity.to_string()``, ensure that the precision argument is also

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.

Do I have to move the change log as part of the manual backport?

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.

I would think merging is at the time of the release, but am not sure...

@pllim
pllim requested review from eteq and mhvk March 24, 2021 20:55

@mhvk mhvk left a comment

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.

Looks good except I don't know the answer to the question about the changelog. I think, though, it should just be backported as is.

@eteq

eteq commented Mar 25, 2021

Copy link
Copy Markdown
Member

I think it's right to leave the changelog as it is here. I'm not going to worry too much about it since towncrier will make this all mott anyway in future versions!

@eteq
eteq merged commit 3f2a946 into astropy:v4.2.x Mar 25, 2021
@pllim
pllim deleted the backport-pr11145 branch March 25, 2021 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants