dmlc / dmlc/dmlc-core

Upgrading array view to span

Open
#446 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
878
Forks
526
Avg merge
12d 19h
Merged PRs (30d)
1

Description

@trivialfis has a PR here https://github.com/dmlc/xgboost/pull/3548 implementing the span class from the cpp core guidelines and the c++20 draft. Ideally we would replace the dmlc-core array_view class with this but dmlc-core currently has no cuda continuous integration testing, which is critical to our implementation. This issue is a placeholder for possibly implementing this in dmlc-core in the future.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the dmlc-core array_view implementation and the linked XGBoost PR #3548 for its span approach. Confirm the required CUDA continuous-integration coverage before defining the replacement scope; done means a compatible span implementation is validated by the relevant CUDA tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.