Replace custom `cachedproperty` with built-in `cached_property`?

wildlife-dynamics / ecoscope

Conservation data analytics

https://ecoscope.io

BSD 3-Clause "New" or "Revised" License

22 stars 9 forks source link

Replace custom `cachedproperty` with built-in `cached_property`? #153

Open cisaacstern opened 4 months ago

cisaacstern commented 4 months ago

I noticed we have our own implementation of cachedproperty

https://github.com/wildlife-dynamics/ecoscope/blob/2fa1e1a1225e0e1518b66e4b7cf32fdafdd781e9/ecoscope/base/utils.py#L76-L81

Is there something specific about this implementation that is lacking from the built-in cached_property? If not, I suggest we use the built-in instead.

atmorling commented 3 months ago

From my quick look at the built-in I don't see anything we're doing that it isn't. @walljcg What do you think?