Penalized B-spline dies while building about splinter

[yrzhanov]

This could be due to the different versions of Eigen - SPLINTER uses 3.2.8, and my libraries are compiled with 3.3.7. However, SPLINTER does not compile with newer Eigen, as line 122 in bsplinebasis.cpp uses InnerIterator which has been made private:
2>F:\D\VariousCodes\splinter\src\bsplinebasis.cpp(122,9): error C2248: 'Eigen::SparseCompressedBase<Eigen::SparseMatrix<double,0,int>>::InnerIterator::InnerIterator': cannot access private member declared in class 'Eigen::SparseCompressedBase<Eigen::SparseMatrix<double,0,int>>::InnerIterator'

Could you please advise how to adapt SPLINTER for newer version of Eigen?
Thank you,
Yuri

[yrzhanov]

Just checked - same data cause no problems when other libraries compiled with later version of Eigen are not present. Is it possible to modify the code so it will work with Eigen 3.3.7?
Thank you,
Yuri

[bgrimstad]

It should definitely be possible to make the code work with Eigen 3.3.7 or a later version. Since I have not read the release notes of the newer Eigen versions, I am not sure how much work it would require. That said, past updates have not required many modifications.

I have noted that, if we ever get around to preparing a new release of SPLINTER, we should consider updating to a newer Eigen version.

[yrzhanov]

Thank you for replying. I tried to recompile OpenCV and PCL with Eigen 3.2.8, but the program still crashes. So meanwhile the only way for me is to write a file with data, spawn another process that calculates a spline and writes the result. My program waits for the completion, reads the output and keeps going. Which is awfully awkward. Looks like the only problem is to replace in SPLINTER the InnerIterator that I mentioned above with non-private member of SparseCompressedBase.

[bowie7070]

I tried forking your repo to integrate with more recent Eigen. The patch I made to deal with the constructor issue I dropped the k argument and changed matrix to vector. Based on the type I assumed that the column vector only needs to be iterated in a single dimension.

5d71820

Let me know if this isn't appropriate. The tests continued to pass but I don't know if this code is covered by tests.

[yrzhanov]

Thank you so much for your efforts. Yes, for our purpose it’s a single dimension. I will give the new version a try right now. Cheers, Yuri From: bowie7070 ***@***.***> Sent: Thursday, July 7, 2022 5:16 AM To: bgrimstad/splinter ***@***.***> Cc: yrzhanov ***@***.***>; Author ***@***.***> Subject: Re: [bgrimstad/splinter] Penalized B-spline dies while building (Issue #132) CAUTION: This email originated from outside of the University System. Do not click links or open attachments unless you recognize the sender and know the content is safe. I tried forking your repo to integrate with more recent Eigen. The patch I made to deal with the constructor issue I dropped the k argument and changed matrix to vector. Based on the type I assumed that the column vector only needs to be iterated in a single dimension. <5d71820> 5d71820 — Reply to this email directly, view it on GitHub <#132 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABCLEFA3Q4EPDAJJQMCVOCDVS2N5BANCNFSM5ZGV2ZOA> . You are receiving this because you authored the thread. <https://github.com/notifications/beacon/ABCLEFACDBRTA4LKJK2VO53VS2N5BA5CNFSM5ZGV2ZOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOIYWBWXQ.gif> Message ID: ***@***.*** ***@***.***> >

[yrzhanov]

Hello Bjarne, Thank you and your colleague for fixing the problem – it took just a single line change – and of course understanding what this line does. Now SPLINTER works with the latest Eigen. I wonder if there is a way to easily detect extremums of the P-spline (points where first derivative is zero)? I don’t have deep understanding how P-spline works (unlike B-spline). Thank you, Yuri From: Bjarne Grimstad ***@***.***> Sent: Tuesday, June 21, 2022 3:08 AM To: bgrimstad/splinter ***@***.***> Cc: yrzhanov ***@***.***>; Author ***@***.***> Subject: Re: [bgrimstad/splinter] Penalized B-spline dies while building (Issue #132) CAUTION: This email originated from outside of the University System. Do not click links or open attachments unless you recognize the sender and know the content is safe. It should definitely be possible to make the code work with Eigen 3.3.7 or a later version. Since I have not read the release notes of the newer Eigen versions, I am not sure how much work it would require. That said, past updates have not required many modifications. I have noted that, if we ever get around to preparing a new release of SPLINTER, we should consider updating to a newer Eigen version. — Reply to this email directly, view it on GitHub <#132 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABCLEFDQGOU2ZHXWVLTW74TVQFS6FANCNFSM5ZGV2ZOA> . You are receiving this because you authored the thread. <https://github.com/notifications/beacon/ABCLEFET33PJKDCXJZIO7UDVQFS6FA5CNFSM5ZGV2ZOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOIU4LZTY.gif> Message ID: ***@***.*** ***@***.***> >