Skip to content

Conversation

@amol-
Copy link
Member

@amol- amol- commented Aug 30, 2021

No description provided.

@github-actions
Copy link

Copy link
Member Author

Choose a reason for hiding this comment

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

This is to fix a bug in printing arrays that lead to

[10,20,30,40   ]

instead of

[10,20,30,40]

when skip_new_lines=True

@amol- amol- marked this pull request as ready for review September 30, 2021 09:02
Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

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

Thanks for this PR!

  1. Can you rebase on master?
  2. Here are some comments.

@amol-
Copy link
Member Author

amol- commented Oct 5, 2021

@pitrou I should have addressed all your comments

Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

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

Thanks for the updates!

@pitrou pitrou closed this in e70fda5 Oct 5, 2021
c0: int16
c1: int32
----
c0: [[1,2,3,4]]
Copy link
Member

Choose a reason for hiding this comment

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

Late comment: I initially found those double square brackets quite confusing (why are there double brackets? Is it a list array of one element?)
(I understand this this is because each column is a ChunkedArray (and the string formatting is coming from there), and so you can have multiple chunks in the column. So also not necessarily something that can be done about this)

ViniciusSouzaRoque pushed a commit to s1mbi0se/arrow that referenced this pull request Oct 20, 2021
Closes apache#11028 from amol-/ARROW-13783

Authored-by: Alessandro Molina <amol@turbogears.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
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