3D Volume initialization from numpy

[steinraf]

Category

• New feature
• Bugfix
• Breaking change
• Refactoring
• Documentation
• Other (please explain)

Description

The current Volume.load_from_numpy method mistakenly initializes a 1D numpy array with values (target_shape[0], target_shape[1], target_shape[2], 3) instead of that shape.

Changelog

• In case bg_value is multidimensional, correctly initialize an array filled by the bg_value
• Adding a test which is not passed by the original version
• Running ruff format

Before your PR is "Ready for review"

• Do you agree to the terms under which contributions are accepted as described in Section 9 the Warp License?
• Have you read the Contributor Guidelines?
• Have you written any new necessary tests?
• Have you added or updated any necessary documentation?
• Have you added any files modified by compiling Warp and building the documentation to this PR (.e.g. stubs.py, functions.rst)?
• Does your code pass ruff check and ruff format --check?

[shi-eric]

Thanks @steinraf. It seems like your pull request makes a lot of unrelated formatting changes, perhaps from a code editor setting that adds trailing commas everywhere, or maybe a max-line-length setting. Can you please update your pull request so that it only has the changes relevant for your bugfix?

[gdaviet]

Thanks @steinraf, this is indeed a bug and the fix looks good to me!
Ideally we would also verify the sampled values in the test, but it looks like the exsiting test_volume_from_numpy was not doing that either, so that's something we can add on our side