search

ACEsuit / ACEfit.jl

Generic Codes for Fitting ACE models
MIT License
7 stars 6 forks source link

BLR_svd lowercase "C" #65

Closed JPDarby closed 1 year ago

JPDarby commented 1 year ago

Should return Dict("C" => coefficients, ...)

wcwitt commented 1 year ago

Curious, I tend to agree, but do you know why this hasn't been a problem before?

JPDarby commented 1 year ago

Curious, I tend to agree, but do you know why this hasn't been a problem before?

Afraid not. I got an error the first time I tried it

cortner commented 1 year ago

merge this too?

wcwitt commented 1 year ago

I hadn't merged because this change broke one of the tests. But I've just fixed it, so merging now...