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

Extend array interface to handle ndarray. #7434

Merged
merged 7 commits into from Nov 16, 2021

Conversation

trivialfis
Copy link
Member

Part of #7405

The ArrayInterface class is extended to support multi-dim array inputs. Previously this class handles only 2-dim (vector is also matrix). This PR specifies the expected dimension at compile-time and the array interface can perform various checks automatically for input data. Also, adapters like CSR are more rigorous about their input. Lastly, row vector and column vector are handled without intervention from the caller.

The `ArrayInterface` class is extended to support multi-dim array inputs. Previously this
class handles only 2-dim (vector is also matrix).  This PR specifies the expected
dimension at compile-time and the array interface can perform various checks automatically
for input data. Also, adapters like CSR are more rigorous about their input.  Lastly, row
vector and column vector are handled without intervention from the caller.
@trivialfis trivialfis added this to 1.6 In Progress in 2.0 Roadmap via automation Nov 14, 2021
@trivialfis trivialfis requested review from hcho3 and removed request for hcho3 November 14, 2021 22:00
@trivialfis trivialfis merged commit 55ee272 into dmlc:master Nov 16, 2021
2.0 Roadmap automation moved this from 1.6 In Progress to 1.6 Done Nov 16, 2021
@trivialfis trivialfis deleted the nd-array-interface branch November 16, 2021 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants