Santiago Soler
Santiago Soler
In the meantime. Do you have a link to the API specification of the NIST data repository? In order to makes our lives easier while maintaining this feature, it's important...
Thanks for opening this PR @leomiquelutti. I assigned myself to review it. I also started running CI so we it can test and check the style of your PR. Feel...
> Actually, there are 4 tests failing: > > * test_derivative_northing_kernel[1] > > * test_derivative_northing_kernel[2] > > * test_derivative_northing_kernel[3] > > * test_laplace_fft > > > Those always failed for...
> Your corrections and suggestions are all good. I am only confused whether I should resolve the conversations or not. And what else I should do, actually. Great! I saw...
> > So, for a PR to be merged, it needs to pass all checks in CI, along with an approved review from one of the developers or maintainers. In...
Indeed, we have the "Require approval for first-time contributors" active in the configuration of GitHub Actions. Here are some [docs about it](https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/enabling-features-for-your-repository/managing-github-actions-settings-for-a-repository#controlling-changes-from-forks-to-workflows-in-public-repositories). I thought that once I approve the first...
> Hi @santisoler is there anything else I should do about this PR? :) Hi @leomiquelutti! No, nothing on your side. I just need to merge it. I'll do it...
> This was my first experience contributing to an open-source project, so I wanted to see that everything went well before starting a new task. Right, that's a very good...
Hi @markgrujic! Thanks for opening this issue. We are aware that a lot of people will benefit from having forward modelling of magnetic fields in Harmonica (see https://github.com/orgs/fatiando/discussions/100 and https://github.com/orgs/fatiando/discussions/102)....
It seems that when running with `macos-13`, the Python setup doesn't work properly: when installing the packages it's using Python 3.12 even if the setup Python action was set to...