search

bloomberg / memray

Memray is a memory profiler for Python
https://bloomberg.github.io/memray/
Apache License 2.0
13.36k stars 397 forks source link

Refactor `HighWatermarkCommand` #325

Closed godlygeek closed 1 year ago

godlygeek commented 1 year ago

This PR implements the cleanup of the HighWatermarkCommand interface that I promised when we were discussing ways to clean up #317 to pass the TemporalAllocationRecord's through the layers more cleanly.

I think this is a huge improvement over what we've got. This is in a reviewable state, I'm only marking it as draft to remind us not to land it until we've landed #317 and rebased this on it.

godlygeek commented 1 year ago

Rebased, now that #317 is landed. Should be good to review at your leisure.

pablogsal commented 1 year ago

Flawless!