Santiago Soler

Results 299 comments of Santiago Soler

I agree that a warning might be a little bit overkill in this case. I would just avoid printing to stout, so we don't clutter the logs. Maybe just print...

Hi Dom! Thanks again for opening this PR. I think we'll aim to merge this PR for SimPEG v0.24.0 (not the next release). Therefore I'll probably delay my review to...

Hi Dom! Most of the comments I made in my previous review were based on the assumption that the Combo class is a collection of `BaseObjectiveFunction`s, not only Scaled ones....

@jcapriot, I could keep working on this for the next release. What do you think about this idea of switching how the mapping gets applied in regs? Regardless of that...

Sounds good. I'll try to keep working on this in the following weeks. If at any point we find a roadblock, I will fall back to the idea of just...

I'd appreciate a second pair of eyes on this, specially on the changes to the gradient and vector regularizations.

I decided to leave the amplitude regularizations our of the scope of this PR. I think we should redesign them a bit, since they not necessarily fit into the `f_m`...

I totally agree. That's a much better solution than this one. I'll revert the previous changes and then apply the one you suggested.

Hi @Mr-Milk. I'm leaving here an issue where we had a similar discussion regarding concurrent downloads and lockfiles: https://github.com/fatiando/pooch/issues/411 In the same spirit of the comments I made on that...