-
Notifications
You must be signed in to change notification settings - Fork 103
[REFACTOR] Clean up chainladder.__init__.py #856
Copy link
Copy link
Labels
Effort > Brief 🐇Small tasks expected to take a few hours up to a couple of days.Small tasks expected to take a few hours up to a couple of days.Impact > Minor 🔷Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Refactor 🔧Code level improvements that restructure existing logic without changing behavior.Code level improvements that restructure existing logic without changing behavior.
Milestone
Metadata
Metadata
Assignees
Labels
Effort > Brief 🐇Small tasks expected to take a few hours up to a couple of days.Small tasks expected to take a few hours up to a couple of days.Impact > Minor 🔷Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Refactor 🔧Code level improvements that restructure existing logic without changing behavior.Code level improvements that restructure existing logic without changing behavior.
Type
Fields
Give feedbackNo fields configured for Task.
Projects
Status
In progress
Description
Currently working on this, I was trying to clean up some repetitive code:
chainladder-python/chainladder/__init__.py
Lines 6 to 18 in 9e833f6
but this revealed other tasks that we'll have to address via multiple PRs. These would include but are not limited to:
Options.get_optionauto_sparseandauto_backendOptions.reset_...for each option.Is your feature request aligned with the scope of the package?
Describe the solution you'd like, or your current workaround.
I will reorganize
Optionswith an init, and set the attributes to be assigned to the instance rather than class.Do you have any additional supporting notes?
No response