-
Notifications
You must be signed in to change notification settings - Fork 751
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added presets for Spinnaker SDK #553
Conversation
Thanks! Have you received permission from FLIR/PGR? Remember that we had to get permission for FlyCapture: https://github.com/bytedeco/javacpp-presets/blob/master/flycapture/src/main/java/org/bytedeco/javacpp/presets/FlyCapture2.java#L23-L26 If you haven't contacted them already, could you also ask them if they could also make the header files and libraries of both FlyCapture and Spinnaker redistributable for the sake of builds only? It would make it much easier for the CI server... |
Do you have a contact person at Point Grey that dealt with FlyCapture permission? |
Not really, no, just support@ptgrey.com. I've forwarded you the last
message I got from them from a thread where I kept you on the Cc list.
|
Request for permission to publish was submitted to Point Grey [PGR #329142]. |
I got clarifications from FLIR / Point Grey. Below are specific points I got alignment from FLIR support:
Additionally we need to do proper credits to FLIR where relevant. I think we are covered here, but I will double check it that we have proper attributions in the code. In summary, we should be fine with the current PR. As for addition of automated build/tests we will need to figure out how to do it without disclosing Spinnaker headers and static libraries. @saudet any ideas how to do that? |
Sounds good! Could you forward to me their response by e-mail for reference? We'll need to provide archives that are accessible only to the CI servers, (which isn't really working for FlyCapture, but) check inside the scripts in the |
@jpsacha Have you received any news from them since? |
Actually, I think you simply haven't forwarded their reply to me yet? It sounds like they were alright with it. In any case, to streamline releases, I will be needing those archives containing the SDK for all platforms. If you have any questions about that, please let me know. We won't be able to make a release until this is done. Thanks for your time! |
@saudet I just sent the original emails from FLIR support to your email. Please confirm that you received it. |
Got it, thanks! Sounds good, so let me know if you have any questions about what to do for the archives... |
I am looking into the archives setup. I will need to get Linux libraries, so fas was only building on Windows... |
I added the archive for testing Spinnaker build. Updated Spinnaker to latest release v.1.15.0.63. |
The build is failing: |
Extract to root to match expected installation location.
Extract to `$HOME/build` then copy to docker root.
Hopefully getting there. Trying to figure out the automated checks process. |
@saudet the latest correction build correctly under Travis |
It's building on linux-x86_64, great! Do you have the build ready for other platforms as well? |
I will check build for Windows 64. Current Java does not support 32-bit, so we can skip those. I am not planning to tinker with ARM. |
So, let's also trim the supported platforms to those 2 in the |
Currently only supporting on linux-x86_64, windows-x86, and windows-x86_64.
@saudet The CI build is now setup for Windows x86, x86_64, and Linux x86_64. Ready to merge. |
The PR contains preset for FLIR's Spinnaker SDK
The current version wraps the C API part. It was tested on Windows x64. Demo code for using the preset is in JavaCV Examples project under
Spinnaker-demo
(initial development is on branchspinnaker
).