Caching spatial_filter

[yfukai]

Caching spatial_filter reduces time to build the rtree. I believe it is safe to share the spatial_filter for the same argument and therefore cache it in the BaseGraph object.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 80.85106% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.44%. Comparing base (e87a375) to head (f30b192).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/tracksdata/utils/_cache.py	73.52%	6 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #215      +/-   ##
==========================================
+ Coverage   88.29%   88.44%   +0.15%     
==========================================
  Files          53       55       +2     
  Lines        3809     3869      +60     
  Branches      661      666       +5     
==========================================
+ Hits         3363     3422      +59     
  Misses        266      266              
- Partials      180      181       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[JoOkuma]

That makes a lot of sense.
Could you add a method to clear the cache? It could be useful if the graph changed.

[yfukai]

Hi @JoOkuma thanks for the suggestion, I added the clear_cache argument!
I think the filter is supposed to automatically change with #159.

[yfukai]

Hi @JoOkuma, does this look okey?

[JoOkuma]

Hi @yfukai, I apologize for missing your recent changes.

When reviewing, I forgot we already had a cache_method implemented. I moved it to its own file (_cache.py) and opened this PR with the changes, yfukai#9

What do you think?

[yfukai]

Thanks @JoOkuma, this looks good to me!