Skip to content

[REVIEW] Add utility to print column#3255

Merged
jrhemstad merged 20 commits intorapidsai:branch-0.11from
ChuckHastings:fea_print_columns
Nov 19, 2019
Merged

[REVIEW] Add utility to print column#3255
jrhemstad merged 20 commits intorapidsai:branch-0.11from
ChuckHastings:fea_print_columns

Conversation

@ChuckHastings
Copy link
Copy Markdown
Contributor

@ChuckHastings ChuckHastings commented Oct 30, 2019

Adds utility to convert a column to a string so it can be printed.

Addresses #3253

Depends on #3232

@ChuckHastings ChuckHastings requested review from a team as code owners October 30, 2019 22:09
@sameerz sameerz added 0 - Backlog In queue waiting for assignment Spark Functionality that helps Spark RAPIDS labels Nov 4, 2019
@harrism
Copy link
Copy Markdown
Member

harrism commented Nov 6, 2019

rerun tests

Copy link
Copy Markdown
Member

@harrism harrism left a comment

Choose a reason for hiding this comment

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

One suggestion.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 7, 2019

Codecov Report

Merging #3255 into branch-0.11 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           branch-0.11   #3255   +/-   ##
===========================================
  Coverage         87.3%   87.3%           
===========================================
  Files               49      49           
  Lines             9215    9215           
===========================================
  Hits              8045    8045           
  Misses            1170    1170

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b6654a4...2a9ca2c. Read the comment docs.

@harrism
Copy link
Copy Markdown
Member

harrism commented Nov 7, 2019

Daily 5pm PST anaconda outage! rerun tests

@harrism harrism added 3 - Ready for Review Ready for review by team libcudf++ and removed 0 - Backlog In queue waiting for assignment labels Nov 7, 2019
Copy link
Copy Markdown
Contributor

@jrhemstad jrhemstad left a comment

Choose a reason for hiding this comment

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

This is looking really good! Love the implementation.

@ChuckHastings
Copy link
Copy Markdown
Contributor Author

Support for timestamps now dependent on #3232

Copy link
Copy Markdown
Contributor

@jrhemstad jrhemstad left a comment

Choose a reason for hiding this comment

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

I'm swooning over how nice the implementation is 😍

@jrhemstad jrhemstad added 5 - Merge After Dependencies and removed 3 - Ready for Review Ready for review by team labels Nov 15, 2019
@jrhemstad
Copy link
Copy Markdown
Contributor

Should not be merged before #3232

@harrism
Copy link
Copy Markdown
Member

harrism commented Nov 19, 2019

@ChuckHastings #3232 is merged, can you resolve conflicts here so we can merge this?

@jrhemstad jrhemstad merged commit ec5d791 into rapidsai:branch-0.11 Nov 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Spark Functionality that helps Spark RAPIDS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants