Santiago Soler

Results 276 comments of Santiago Soler

True. I think it's a good compromise. An alternative would be to define the receiver list as an optional argument. I don't have strong feelings toward any of these options.

I wouldn't like to change order of arguments, that's why I decided to make `receiver_list` as required from the start. I also agree with the idea of not setting default...

I think we could have some discussions around it and after deciding a path to follow we could label this issue as a good-first-issue so new contributors can help implementing...

@jcapriot I added a new `typing` module with the `RandomSeed` type alias. I also added a new documentation page for it. I noticed that it was easier to just set...

I kept using `SimPEG` in the API reference while including the documentation for the new `typing` module in order to follow the current convention in `main`. I opened #1437 to...

Hi @kehrl-kobold! Thanks a lot for opening this PR. From just reading the code, those changes look right. Did you run into this bug while using this function? Do you...

Thanks @thibaut-kobold! You're right, the tutorial will be rerun after we get the updated merged. And in the next release the updated docs will be deployed to the website. I...

Sounds great! We don't have any rush to merge this, since it's not super critical, so feel free to take your time. > Would [this](https://github.com/simpeg/simpeg/blob/main/tests/em/static/test_DC_Utils.py) be the correct file for...

Thanks @kehrl-kobold! I've just updated this branch to get the latest changes in `main`. I'll merge it after CI passes. Thanks again for spotting this bug and also fixing it!

Merged! 🚀 Thanks again for your contribution @kehrl-kobold! Feel free to open a PR to add yourself to the `AUTHORS.rst` if you desire.