Skip to content
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.

Sign up for GitHub

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

Jump to bottom

WIP: Add Java bindings for Android #173

Closed
wants to merge 1 commit into from
Closed

WIP: Add Java bindings for Android #173

wants to merge 1 commit into from

Conversation

Meakk
Copy link
Contributor

@Meakk Meakk commented Dec 21, 2021 •
edited
Loading

  • Build VTK in CI in 64-bits ARM and x86 architectures
  • Build libf3d with minimal modules in CI
  • Add JNI wrapper in CI (expose libf3d-java.so artifacts)

Will be done later:

  • Create Java or Kotlin interface
  • Create Gradle build system
  • Generate .apk (expose in CI artifacts)
  • Add Exodus module
  • Add Assimp module
  • Add OCCT module
  • Add 32-bits architectures
@Meakk Meakk self-assigned this Dec 21, 2021
@Meakk Meakk force-pushed the android branch 28 times, most recently from fb3706c to 734b49b Compare December 21, 2021 23:13
@mwestphal mwestphal mentioned this pull request Dec 25, 2021
Closed
10 tasks
@mwestphal mwestphal mentioned this pull request May 20, 2022
Open
4 tasks
@Meakk Meakk force-pushed the android branch 2 times, most recently from 40d2ae0 to 6282ed1 Compare June 2, 2022 15:19
@f3d-app f3d-app deleted a comment from codecov bot Jun 2, 2022
@codecov
Copy link

codecov bot commented Jun 2, 2022 •
edited
Loading

Codecov Report

Merging #173 (66d6a59) into master (52c68e7) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 66d6a59 differs from pull request most recent head e12b239. Consider uploading reports for the commit e12b239 to get more accurate results

@@            Coverage Diff             @@
##           master     #173      +/-   ##
==========================================
+ Coverage   94.87%   94.89%   +0.02%     
==========================================
  Files          91       91              
  Lines        4524     4545      +21     
==========================================
+ Hits         4292     4313      +21     
  Misses        232      232
Impacted Files Coverage Δ
.../VTKExtensions/Applicative/vtkF3DObjectFactory.cxx 100.00% <ø> (ø)
library/VTKExtensions/Rendering/vtkF3DRenderer.cxx 99.74% <ø> (ø)
library/public/window.h 100.00% <ø> (ø)
library/src/window_impl.cxx 100.00% <100.00%> (ø)
...xtensions/Rendering/vtkF3DRendererWithColoring.cxx 98.26% <0.00%> (-0.01%) ⬇️
library/public/types.h 100.00% <0.00%> (ø)
library/public/camera.h 100.00% <0.00%> (ø)
library/src/camera_impl.cxx 100.00% <0.00%> (ø)
... and 2 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more
Meakk
Meakk commented Jun 3, 2022
View reviewed changes
library/public/interactor.h Outdated Show resolved Hide resolved
@Meakk Meakk force-pushed the android branch 2 times, most recently from 9f20614 to 350b8aa Compare June 14, 2022 21:46
@Meakk
Copy link
Contributor Author

Meakk commented Aug 25, 2022

Current status:

Screen_Recording_20220825-214348_F3D.mp4
@mwestphal
Copy link
Contributor

nice, the zoom is a bit janky, is it a FPS issue or interaction issue ?
@Meakk
Copy link
Contributor Author

Meakk commented Aug 26, 2022

It's an interaction issue
@Meakk Meakk force-pushed the android branch 5 times, most recently from 66d6a59 to 6864b5f Compare August 30, 2022 22:05
@Meakk Meakk mentioned this pull request Aug 30, 2022
Merged
@Meakk
Copy link
Contributor Author

Meakk commented Sep 3, 2022

Android build is merged here: #382
New required APIs here: #383

This PR will now focus on java bindings
@Meakk Meakk changed the title WIP: Add Android build Sep 3, 2022
@Meakk
Copy link
Contributor Author

Meakk commented Sep 3, 2022

Actually, closing this one for #390
@Meakk Meakk closed this Sep 3, 2022
@Meakk Meakk deleted the android branch September 3, 2022 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@Meakk @mwestphal