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.

Sign up for GitHub

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

Jump to bottom

#93a Fix TreeCorr imports and 1d arrays in FITS table writing #93

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

#93a Fix TreeCorr imports and 1d arrays in FITS table writing #93

wants to merge 7 commits into from

Conversation

msimet
Copy link
Owner

@msimet msimet commented Nov 8, 2016

Hi folks,

Two cleanup items in this PR:

  • We say in the README that TreeCorr is a good idea, but not required. That wasn't actually true. This fixes the rogue import unprotected by a try block.
  • The FITS standard allows for columns where each row of the column is actually a 1D array. The way I coded up to turn numpy arrays into FITS tables didn't allow for that, though. This fix does allow it.

msimet added a commit that referenced this pull request Jun 21, 2018
@msimet
Merge branch 'master' into #93
7dd5362
msimet added a commit that referenced this pull request Jul 12, 2018
@msimet
Merge branch 'master' into #93
966f332
msimet added a commit that referenced this pull request Jul 14, 2018
@msimet
Merge branch 'master' into #93
0519cdb
msimet added a commit that referenced this pull request Nov 17, 2018
@msimet
Merge branch 'master' into #93
7efc154
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
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@msimet