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.

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

C++ minimal CSV reader #2791

Merged
merged 10 commits into from
Oct 13, 2022
Merged

Conversation

seunghwak
Copy link
Contributor

Clean-up the C++ test graph generation code and add a minimal C++ CSV (this can read edge lists only in a strict format this reader is expecting) reader.

The CSV reader is added to support edge list with non-stand vertex IDs (e.g. negative vertex IDs).

@seunghwak seunghwak requested review from a team as code owners October 11, 2022 20:07
@seunghwak seunghwak self-assigned this Oct 11, 2022
@seunghwak seunghwak added feature request New feature or request 2 - In Progress non-breaking Non-breaking change labels Oct 11, 2022
@seunghwak seunghwak added this to the 22.12 milestone Oct 11, 2022
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (branch-22.12@9ee03f2). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@               Coverage Diff               @@
##             branch-22.12    #2791   +/-   ##
===============================================
  Coverage                ?   60.45%           
===============================================
  Files                   ?      111           
  Lines                   ?     6514           
  Branches                ?        0           
===============================================
  Hits                    ?     3938           
  Misses                  ?     2576           
  Partials                ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ChuckHastings
Copy link
Collaborator

@gpucibot merge

@rapids-bot rapids-bot bot merged commit ede59d5 into rapidsai:branch-22.12 Oct 13, 2022
@seunghwak seunghwak deleted the fea_cpp_csv_reader branch October 20, 2022 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants