issue_comments: 671475798

This data as json

html_url issue_url id node_id user created_at updated_at author_association body reactions performed_via_github_app issue
https://github.com/pydata/xarray/issues/4300#issuecomment-671475798 https://api.github.com/repos/pydata/xarray/issues/4300 671475798 MDEyOklzc3VlQ29tbWVudDY3MTQ3NTc5OA== 1217238 2020-08-10T17:06:11Z 2020-08-10T17:06:11Z MEMBER

+1 for just wrapping the existing functionality in SciPy for now. If we want a version of curve_fitthat supports dask, I would suggest implementingcurve_fit` with dask first, and then using that from xarray.

I am OK with using inspect from the standard library for determining default parameter names. inspect.signature is reasonably robust. But there should definitely be an optional argument for setting parameter names explicitly.

 
{
    "total_count": 2,
    "+1": 2,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   671609109