llnl / llnl/axom

Array Updates

Open
#633 11 comments 1 reaction 1 assignee View on GitHub

Nobody has claimed this yet.

Core GPU Reviewed
Dominant language
C++
Stars
196
Forks
34
Avg merge
4d 1h
Merged PRs (30d)
11

Description

From #627

I envision making the following changes in each PR (roughly in order)

  • Extending Array to multiple dimensions - #627
  • Replacing MCArray with Array (now that multiple dimensions are possible - #657
  • Adding a View (ArrayView?) class that provides indexing/access to a pointer it does not own, while removing the external data feature from Array - #678
  • Adding GPU support via a template parameter for the memory space of the Array/ArrayView - this needs to include a dynamic option so the memory space can be set at runtime - #695

Other (smaller?) feature requests/tasks:

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.