Skip to content
This repository was archived by the owner on Feb 26, 2025. It is now read-only.

Conversation

1uc
Copy link
Collaborator

@1uc 1uc commented May 1, 2023

This commit allows users to specify the memory dataspace. Previously, we forced the memspace to be 1D, because in general a hyperslab selection isn't a multi-dimensional block anymore.

We now allow users to specify the memory dataspace, which must be simple; but can be multi-dimensional.

This commit allows users to specify the memory dataspace. Previously, we
forced the memspace to be 1D, because in general a hyperslab selection
isn't a multi-dimensional block anymore.

We now allow users to specify the memory dataspace, which must be
simple; but can be multi-dimensional.
@codecov
Copy link

codecov bot commented May 1, 2023

Codecov Report

Merging #740 (784eb9b) into master (daad5ee) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #740   +/-   ##
=======================================
  Coverage   82.47%   82.47%           
=======================================
  Files          67       67           
  Lines        4543     4543           
=======================================
  Hits         3747     3747           
  Misses        796      796           
Impacted Files Coverage Δ
include/highfive/bits/H5Slice_traits.hpp 95.52% <ø> (ø)
include/highfive/bits/H5Slice_traits_misc.hpp 90.08% <100.00%> (ø)

@1uc 1uc marked this pull request as ready for review May 1, 2023 15:18
Copy link
Contributor

@pramodk pramodk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks straightforward change!

@1uc 1uc merged commit 493b5fe into master May 3, 2023
@1uc 1uc deleted the 1uc/hyperslab-memspace branch May 3, 2023 06:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants