Skip to content

Representations repr fix - #5896

Closed
mhvk wants to merge 2 commits into
astropy:masterfrom
mhvk:representations-repr-fix
Closed

Representations repr fix#5896
mhvk wants to merge 2 commits into
astropy:masterfrom
mhvk:representations-repr-fix

Conversation

@mhvk

@mhvk mhvk commented Mar 21, 2017

Copy link
Copy Markdown
Contributor

fixes #5889

Now, instead of TypeError, this works:

CartesianRepresentation(np.arange(27).reshape(3,3,3), unit='m').T
# <CartesianRepresentation (x, y, z) in m
#     [[( 0.,   9.,  18.), ( 3.,  12.,  21.), ( 6.,  15.,  24.)],
#      [( 1.,  10.,  19.), ( 4.,  13.,  22.), ( 7.,  16.,  25.)],
#      [( 2.,  11.,  20.), ( 5.,  14.,  23.), ( 8.,  17.,  26.)]]>

@mhvk mhvk added this to the v1.3.2 milestone Mar 21, 2017
@mhvk
mhvk force-pushed the representations-repr-fix branch from a887373 to 7fa03ac Compare March 21, 2017 20:18
@mhvk mhvk mentioned this pull request Mar 21, 2017
Closed
@mhvk

mhvk commented Mar 21, 2017

Copy link
Copy Markdown
Contributor Author

Oops sorry, only now see #5891 - will make PR to that instead...

@mhvk mhvk closed this Mar 21, 2017
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

Bug coordinates

Projects

None yet

Milestone

v1.3.2

Development

Successfully merging this pull request may close these issues.

__repr__ method of BaseRepresentation can fail

1 participant

@mhvk