Catch exception in WeightedLassoCV and re-direct to WeightedMultiTaskLassoCV
Open
@moprescu is already working on this.
Since Nov 26, 2019.
bug
- Dominant language
- Jupyter Notebook
- Stars
- 4.8k
- Forks
- 827
- PR merge metrics
- No merged PRs in 30d
Description
When one calls WeightedLassoCV with a multi-dimensional y then it throws an error
ValueError: For multi-task outputs, use MultiTaskLassoCV
This should be changed to:
ValueError: For multi-task outputs, use WeightedMultiTaskLassoCV
One way to do this would be to wrap the following line:
https://github.com/microsoft/EconML/blob/73964d37965e98b22ce423cdfed6adbe26e1f448/econml/sklearn_extensions/linear_model.py#L414
inside a try-catch statement and cathc the above exact value error message. Then throw the new error message when that happens.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.