[Option Appraisal Module] [Package 1: Measures] Split 0: Move old measures to _legacy module#1274
[Option Appraisal Module] [Package 1: Measures] Split 0: Move old measures to _legacy module#1274
Conversation
|
Excellent! Quick question for completness: would this lead to/allow us supporting the use of the current |
|
Yes, that's exactly the idea! This PR makes it so that everything that requires Measure and MeasureSet before the refactor go and fetch them from _legacy_measure (which contains the code before the refactor), including CostBenefit. Once we don't want these any more, we can remove them and the _legacy_measure folder. The code that requires the new objects will import from the original measure folder, containing the new code. |
|
What do you think about the tutorials for the old More specifically:
|
|
I went ahead.
This PR is open for review! |
peanutfun
left a comment
There was a problem hiding this comment.
Looks good to me! Only one thing: Please add a link to the new "Adaptation appraisal guides" menu to the Attention block in the old tutorial.
Would it make sense to add a deprecation warning to the old Measures class or would that be too confusing?
This PRs preps the field for the reworks of the Measure / MeasureSet classes.
Because the design completely changes, tests and code using the legacy code will fail.
To avoid that while gradually pushing the new code and before the new option appraisal module is fully ready, this PR moves the old code from
measuresto_legacy_measuresmodule, and import from there.PR Author Checklist
develop)PR Reviewer Checklist