danieldresser-ie
commented
2 weeks ago I've switched to using variableSize, and added some more info to the comment ( looks like there isn't any good reason why CurvesAlgo.resamplePrimitiveVariable is doing weird things, but that's a fix for another day ).
Should be good to merge.
Not sure how much periodic curves are ever actually used, but as long as our curves type supports them, we probably shouldn't output bogus data for them.