search

fatiando / harmonica

Forward modeling, inversion, and processing gravity and magnetic data
https://www.fatiando.org/harmonica
BSD 3-Clause "New" or "Revised" License
211 stars 69 forks source link

Add covariance, change fit data, more docs to Euler Deconvolution #519

Closed leouieda closed 2 months ago

leouieda commented 3 months ago

The covariance matrix will be needed to filter solutions in the windowed approach. Change the fit input to receive all the data in a single tuple, like the coordinates. This is more compatible with the rest of Fatiando. Added the checks for fit input and standardizing array shapes so inputs can be n-dimensional. Copied over some of the docstrings from the legacy Fatiando implementation, which expands the theory and provides links to good papers to read.

leouieda commented 3 months ago

@Souza-junior I remembered after merging #493 that we had a lot of docs in the old Fatiando that could be copied over. The extra lines at the start of the fit method are used throughout Fatiando to help make all the inputs tuples of 1d arrays, which is easier to handle. They're very useful to allow inputs to be of any shape.

leouieda commented 3 months ago

@Souza-junior could you take a quick look and see what you think?

Souza-junior commented 3 months ago

@leouieda I'll take a closer look into it.

Souza-junior commented 2 months ago

@leouieda it looks good!