Closed msridhar closed 1 year ago
I found #132 a bit hard to understand because I couldn't understand what exactly TrackerNode was for. Here I renamed it to ImpactedRegion and added some documentation, which I think is helpful. There should be no functional change here.
TrackerNode
ImpactedRegion
@nimakarimipour please take a look. If it looks ok, maybe we can land this before #132?
I found #132 a bit hard to understand because I couldn't understand what exactly
TrackerNode
was for. Here I renamed it toImpactedRegion
and added some documentation, which I think is helpful. There should be no functional change here.