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

Fix structs column description in dev docs #8318

Merged
merged 4 commits into from
May 25, 2021

Conversation

isVoid
Copy link
Contributor

@isVoid isVoid commented May 21, 2021

Currently structs column section of developer documentation mentions there's an offset column tied to its layout. This is not true and this PR fixes that.

@isVoid isVoid requested a review from a team as a code owner May 21, 2021 19:29
@isVoid isVoid requested review from harrism and rgsl888prabhu May 21, 2021 19:29
@github-actions github-actions bot added the libcudf Affects libcudf (C++/CUDA) code. label May 21, 2021
@isVoid isVoid added doc Documentation bug Something isn't working non-breaking Non-breaking change and removed bug Something isn't working labels May 21, 2021
cpp/docs/DEVELOPER_GUIDE.md Outdated Show resolved Hide resolved
cpp/docs/DEVELOPER_GUIDE.md Outdated Show resolved Hide resolved
@mythrocks
Copy link
Contributor

Thanks for fixing this. I'm not sure how I missed this when reviewing the last doc PR. I guess I concentrated on the struct null-mask bit.

@codecov
Copy link

codecov bot commented May 21, 2021

Codecov Report

❗ No coverage uploaded for pull request base (branch-21.06@59d8d5e). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.06    #8318   +/-   ##
===============================================
  Coverage                ?   82.88%           
===============================================
  Files                   ?      105           
  Lines                   ?    17874           
  Branches                ?        0           
===============================================
  Hits                    ?    14814           
  Misses                  ?     3060           
  Partials                ?        0           

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 59d8d5e...88f92bb. Read the comment docs.

@isVoid isVoid added 5 - Ready to Merge Testing and reviews complete, ready to merge 3 - Ready for Review Ready for review by team and removed 5 - Ready to Merge Testing and reviews complete, ready to merge labels May 24, 2021
@nvdbaranec nvdbaranec self-requested a review May 25, 2021 14:23
@nvdbaranec
Copy link
Contributor

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 6db757b into rapidsai:branch-21.06 May 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team doc Documentation libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants