Tharwat Posted March 26, 2021 Posted March 26, 2021 @ronjonp I see that you have changed your mind about the use of vl-remove function. Quote
ronjonp Posted March 26, 2021 Posted March 26, 2021 (edited) 8 minutes ago, Tharwat said: @ronjonp I see that you have changed your mind about the use of vl-remove function. @Tharwat Yup .. but I think it will work either way. Since we're sorting by distance within the loop removing the first then second sorted item is simpler IMO. Although if you have an outlier without a close matching pair this method will probably bork Edited March 26, 2021 by ronjonp Quote
Tharwat Posted March 26, 2021 Posted March 26, 2021 Yes I agree, but now with your different 'posted' approach and the mods you did over my codes the OP or any reviewer of this thread would think that yours is the correct one and mine has some issues. Don't you think so? Quote
ronjonp Posted March 26, 2021 Posted March 26, 2021 (edited) 30 minutes ago, Tharwat said: Yes I agree, but now with your different 'posted' approach and the mods you did over my codes the OP or any reviewer of this thread would think that yours is the correct one and mine has some issues. Don't you think so? Your code works fine and my intention was not to belittle your effort, sorry if you feel this way. I think the the OP is already using your code as a solution. My offering was just an alternative to learn from ( or not ). Edited March 26, 2021 by ronjonp Quote
Tharwat Posted March 26, 2021 Posted March 26, 2021 1 hour ago, ronjonp said: Your code works fine and my intention was not to belittle your effort, sorry if you feel this way. I think the the OP is already using your code as a solution. My offering was just an alternative to learn from ( or not ). Agree with you, and I do like such fruitful discussions. Quote
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.