So it's still wrong I am fairly certain however, it's not worse than what's in (curve's already 0-to-1) so there is no reason to hold-this change, I can open another issue or put a little…
So normally this code is updated in my change but my projects.blender.org is all jacked-up.
(Is this a GitLab instance, GitHub instance ? Anyways why don't we use Subversion, that'd be fixed ? …
I'm resolving this one even though I'm not putting r_nu.
Do you want me to do that instead, @mod_moder ?
Rather, I have used a Span and used r_ in the static functions in the C++ file.
Yeah from create_curve() just above, with which it could be merged.. Food for thoughts
I have moved it in its own static const variable (probably should be using constexpr ? Hey..)
(I think it's cool. It helps a great deal cleaning-up unrequired dependencies.)
OK. I think the compiler will say no for the const in the static cast, lets see
Yes: it should be a size_t. However I understood you were not taking a liking to it :P.
Thank you ! The code is not correct here either: I think it should be the values of the 1st and last knot
Thank you! I shamelessly copied that code, but was going to make a point of using the size of param instead!
Oh I did not write that.. It's going to have to be a bad merge, sloppy copy-paste or something.
Thank you I'll find what happened here when I go over all your comments
Yes this is wrong BTW, thank you!
I was a bit overly-excited when I removed the WIP but I think this needs being addressed: We will have 3 coordinate point (and 1 will be implied.
I don't…
☝️ As per above! Ready to check-in for this issue! I am moving onto the FreeCAD side-of-things now :] o/
Finally! (I was mixed-up in my last comment: it is bezierthat is being used in the function BKE_nurb_knot_calc_u. endpoint is…
... I can ...
The offer alone is a huge cheer-up. Thank you ! Sadly, we've got all the files we need (both working and not. I think.)
What I'm leaning toward we should create…