GradcheckError with deepwave v0.0.13 on nixpkgs about deepwave

[AtilaSaraiva]

Merged

[AtilaSaraiva]

Seems like the values of the tensors are quite similar, but the test fails nevertheless. I prefer waiting for the next release than updating to the latest one and running the risk of breaking the users projects. Either way, thank you for your amazing project, it surprised me when I discovered.

[ar4]

Oh wow, I didn't know that you were doing that. Thank you! Thank you also for your care in making sure that the tests pass and for reporting this problem. All of the tests pass on my computer and on GitHub Actions, but I must have set the tolerances a bit too tight for it to pass everywhere. I plan to have another release within about two weeks, so I will adjust the tolerances on that test before then. Would it be helpful if I committed the new version as a branch on GitHub and notified you so that you could check it before I release it?

[AtilaSaraiva]

Oh no problem! I love to support geophysics related project by packaging then, I've done it to devito, pyrevolve and segyio too.
oh about the branch pre-release, that would help a lot! I don't know much about writing github actions workflows, but it is also possible to create one that does this test with nix. I will talk to my nix friends so I can figure this out.

[ar4]

I have now released v0.0.14. The Nixpkgs tests pass, so hopefully there won't be any problems this time.

[AtilaSaraiva]

Ohh thanks, I will bump it right away!