Joe Wallwork
Joe Wallwork
Got round to updating this branch. The simplest thing to do was to merge `main` into it, so it should be merged with squash.
As discussed in today's meeting, we should check what happens if an operator that hasn't been implemented is used. I switched from `Q = 3 * a ** 3 -...
Merged in the Windows fix from #207.
> Good idea. Would it be worth having an alias for it as well called `torch_array_from_tensor`? Sure, I guess `torch_array_to_tensor` would then complete the set.
> I have a question about the update to example 1 - presumably the use of `torch_tensor_to_array` is redundant here if you had not also changed the `out_data` to `pointer`?...
This is now ready for review, with thanks to @TomMelt for input on the C++ side.
> @jwallwork23 Isn't this equivalent to the Loop.independent_iterations method? (I think we considered this name for that method but decided the independent iterations could be more generic) Oh good, if...
Thanks again for your review on this @arporter. I haven't fully tested the new functionality of `ParallelLoopTrans` but I think enough of this PR is invalid that I'll close it...
> Could I be a bit cheeky and request a fix for a docstring formatting error from a previous PR? Building the Ref. Guide now gives: > > ``` >...
> Ups, I missed this on my review. I think the issue comes from missing the starting colon in the `:returns:` and `:rtype:` tags. It is also wrong in the...