Skip to content

fix: pass attrs parameter to wrap_layout in from_*#3687

Merged
ianna merged 2 commits intoscikit-hep:mainfrom
DylanModesitt:dcm/fix-attrs-in-from-functions
Oct 24, 2025
Merged

fix: pass attrs parameter to wrap_layout in from_*#3687
ianna merged 2 commits intoscikit-hep:mainfrom
DylanModesitt:dcm/fix-attrs-in-from-functions

Conversation

@DylanModesitt
Copy link
Copy Markdown
Contributor

Closes: #3686

This is a pretty small bug, but I noticed it and figured it looked like a trivial fix.

@DylanModesitt DylanModesitt changed the title fix pass attrs parameter to wrap_layout in from_* fix: pass attrs parameter to wrap_layout in from_* Oct 17, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.70%. Comparing base (b749e49) to head (220035a).
⚠️ Report is 451 commits behind head on main.

Additional details and impacted files
Files with missing lines Coverage Δ
src/awkward/operations/ak_from_cupy.py 85.71% <ø> (ø)
src/awkward/operations/ak_from_dlpack.py 63.33% <ø> (ø)
src/awkward/operations/ak_from_jax.py 100.00% <ø> (ø)
src/awkward/operations/ak_from_numpy.py 100.00% <ø> (ø)

... and 193 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link
Copy Markdown

The documentation preview is ready to be viewed at http://preview.awkward-array.org.s3-website.us-east-1.amazonaws.com/PR3687

Copy link
Copy Markdown
Member

@ianna ianna left a comment

Choose a reason for hiding this comment

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

@DylanModesitt - Thanks for fixing it!

@ianna ianna added the pr-next-release Required for the next release label Oct 20, 2025
@ianna ianna merged commit 0349a55 into scikit-hep:main Oct 24, 2025
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-next-release Required for the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

attrs not preserved in ak.from_numpy

2 participants