Tue 21 Mar 2017 06:20:06 PM UTC, comment #4:
Hi,
It wasn't a problem with argument checking but actually a bug on the origina code.
When function cross is used, the dimension is not specified (should be 2) and then when the input is 3x3 the 1st dimension was used instead of the 2nd.
I grep for 'cross[[:space:]]\([[:alum:]]+,[[:space:]][[:alum:]]+\)' and found some functions that still call cross without specifying dimension, which are potential bugs. However these functions are not yet added to geometry.
@devs: Please close this bug.
|
Mon 20 Feb 2017 03:16:09 PM UTC, original submission:
Hi all,
I noticed an error with the result of the intersectLinePlane function in the geometry package when the inputs are 3 lines and 3 planes. This code reproduces the issue:
Looking at the function code, the problem seems to lie with line 82. The function should always be done along dim 2, but when a 3x3 matrix is provided it incorrectly is done on dim 1. Adding dim argument of 2 corrects the issue.
I attached the code to reproduce and the corrected function. Thanks
|