Skip to content

Change way slicing of recarray is done to help numpy 1.9 - #6966

Merged
bsipocz merged 1 commit into
astropy:masterfrom
mhvk:fits_rec_further_small_change
Dec 11, 2017
Merged

bsipocz merged 1 commit into
astropy:masterfrom
mhvk:fits_rec_further_small_change

Conversation

@mhvk

@mhvk mhvk commented Dec 11, 2017

Copy link
Copy Markdown
Contributor

Tentative fix for numpy 1.9 for the 2.0.3 branch; probably somewhat more logical than my original anyway, so good to go in master too. I think this can be merged if tests on 2.0.3 pass with it.

@mhvk mhvk added this to the v2.0.3 milestone Dec 11, 2017
@mhvk
mhvk requested a review from bsipocz December 11, 2017 15:44
@astropy-bot

astropy-bot Bot commented Dec 11, 2017 •

Copy link
Copy Markdown

Hi there @mhvk 👋 - thanks for the pull request! I'm just a friendly 🤖 that checks for issues related to the changelog and making sure that this pull request is milestoned and labeled correctly. This is mainly intended for the maintainers, so if you are not a maintainer you can ignore this, and a maintainer will let you know if any action is required on your part 😃.

Everything looks good from my point of view! 👍

If there are any issues with this message, please report them here.

@bsipocz

bsipocz commented Dec 11, 2017

Copy link
Copy Markdown
Member

@bsipocz bsipocz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

passes on 2.0.x

@bsipocz
bsipocz merged commit 89276fe into astropy:master Dec 11, 2017
@mhvk

mhvk commented Dec 11, 2017

Copy link
Copy Markdown
Contributor Author

Great!

@mhvk
mhvk deleted the fits_rec_further_small_change branch December 11, 2017 17:27
bsipocz added a commit that referenced this pull request Dec 11, 2017
Change way slicing of recarray is done to help numpy 1.9
@saimn

saimn commented Dec 11, 2017

Copy link
Copy Markdown
Contributor

Thanks for the quick fix !

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