Skip to content

We should reimplement our jagged tensor reductions using scatter_reduce_ #575

@swahtz

Description

@swahtz

We should reimplement our jagged tensor reductions using scatter_reduce_ rather than the current implementation which uses naive reduction with atomics.

I'm approving the PR to get rid of the dependency, but as a follow up, could you also change the JaggedReduce implementation to use scatter_reduce_?

Originally posted by @fwilliams in #571 (review)

Metadata

Metadata

Assignees

No one assigned

    Labels

    core libraryCore fVDB library. i.e. anything in the _Cpp module (C++) or fvdb python moduletriageNeeds team review

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions