jrkerns / pylinac

An image analysis library for medical physics
https://pylinac.readthedocs.io/en/latest/
MIT License
157 stars 99 forks source link

Remove Unused Dependency: Cached-property #461

Closed gdrosos closed 1 year ago

gdrosos commented 1 year ago

Summary

This pull request removes the unused dependency cached-property from the poetry.lock and requirements-ram.txt configuration files. The removal of this dependency is a finding from ongoing research aimed at identifying and eliminating code bloat within software projects.

Rationale

The cached-property library was first introduced to the project in f60d4e0. However, it was subsequently replaced by functools.cached_property in 42ac098. Despite its replacement in the source code, cached-property has persisted as a requirement in some of the project's dependency files. The removal of this now redundant dependency will reduce the app's footprint and streamline dependency management.

Changes

Impact

gdrosos commented 1 year ago

Hello @jrkerns,

I hope this message finds you well. We are currently engaged in a research project focused on optimizing projects by identifying and eliminating unnecessary dependencies. Your insights and expertise as a maintainer would greatly contribute to our research efforts.

Could you please take a moment to review these changes? Your input would be highly valuable to us.

Thank you for your consideration!