physicsnemo icon indicating copy to clipboard operation
physicsnemo copied to clipboard

DLWP Indexing and memory consumption fix

Open daviddpruitt opened this issue 7 months ago • 3 comments

PhysicsNeMo Pull Request

Description

Fixes indexing issues in couplers and alleviates issue with large CPU memory consumption in dataloader

Checklist

  • [x] I am familiar with the Contributing Guidelines.
  • [x] New or existing tests cover these changes.
  • [x] The documentation is up to date with these changes.
  • [ ] The CHANGELOG.md is up to date with these changes.
  • [ ] An issue is linked to this pull request.

Dependencies

daviddpruitt avatar Apr 28 '25 16:04 daviddpruitt

/blossom-ci

loliverhennigh avatar May 01 '25 18:05 loliverhennigh

Don't have much to comment on with this. If it passes CI I would say merge unless there is something specific you want me to look at @daviddpruitt

loliverhennigh avatar May 01 '25 18:05 loliverhennigh

@daviddpruitt has this been covered by #879?

pzharrington avatar May 09 '25 00:05 pzharrington

Relevant changes included in #879

daviddpruitt avatar Jul 21 '25 23:07 daviddpruitt