8000 Replacing bullet-native with Minie by neph1 · Pull Request #547 · jMonkeyEngine/sdk · GitHub
[go: up one dir, main page]
Skip to content

Replacing bullet-native with Minie #547

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
Merged
merged 6 commits into from
Nov 24, 2023
Merged

Replacing bullet-native with Minie #547

merged 6 commits into from
Nov 24, 2023

Conversation

neph1
Copy link
Contributor
@neph1 neph1 commented Nov 18, 2023 •
edited
Loading

Removing blender module (will probably have to revert)
Removing xbuf,blender dependency

Fixes #536

@neph1
Replacing bullet-native with Minie
843841a 
Removing blender module (will probably have to revert)
Removing xbuf,blender dependency
@neph1
Copy link
Contributor Author
neph1 commented Nov 18, 2023 •
edited
Loading

jme3utilities is part of the Minie library. It resides in core-baselibs, just like bullet-native used to.
This is the stacktrace:
java.lang.NoClassDefFoundError: com/jme3/math/Vector3f
at jme3utilities.Validate.finite(Validate.java:151)
at com.jme3.bullet.CollisionSpace.(CollisionSpace.java:156)
at com.jme3.bullet.PhysicsSpace.(PhysicsSpace.java:275)
at com.jme3.bullet.BulletAppState.createPhysicsSpace(BulletAppState.java:717)
at com.jme3.bullet.BulletAppState.startPhysics(BulletAppState.java:643)
at com.jme3.bullet.BulletAppState.stateAttached(BulletAppState.java:884)
at com.jme3.app.state.AppStateManager.attach(AppStateManager.java:147)
at com.jme3.gde.core.scene.SceneApplication$11.call(SceneApplication.java:689)
at com.jme3.app.AppTask.invoke(AppTask.java:147)
at com.jme3.app.LegacyApplication.runQueuedTasks(LegacyApplication.java:759)
at com.jme3.app.LegacyApplication.update(LegacyApplication.java:775)
at com.jme3.gde.core.scene.SceneApplication.update(SceneApplication.java:296)
at com.jme3.system.awt.AwtPanelsContext.updateInThread(AwtPanelsContext.java:237)
at com.jme3.system.awt.AwtPanelsContext.access$200(AwtPanelsContext.java:46)
at com.jme3.system.awt.AwtPanelsContext$AwtPanelsListener.update(AwtPanelsContext.java:80)
at com.jme3.system.lwjgl.LwjglOffscreenBuffer.runLoop(LwjglOffscreenBuffer.java:126)
at com.jme3.system.lwjgl.LwjglOffscreenBuff 8000 er.run(LwjglOffscreenBuffer.java:160)
at java.base/java.lang.Thread.run(Thread.java:833)
Caused by: java.lang.ClassNotFoundException: com.jme3.math.Vector3f starting from ModuleCL@58f8fbb7[com.jme3.gde.core.libraries] with possible defining loaders [ModuleCL@e75e040[com.jme3.gde.core.baselibs]] and declared parents ]
at org.netbeans.ProxyClassLoader.doFindClass(ProxyClassLoader.java:225)
at org.netbeans.ProxyClassLoader.loadClass(ProxyClassLoader.java:139)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:520)
... 18 more
Caused by: java.lang.ClassNotFoundException: com.jme3.math.Vector3f
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641)
at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:520)
at org.netbeans.ProxyClassLoader.doFindClass(ProxyClassLoader.java:223)
... 20 more

@neph1
Copy link
Contributor Author
neph1 commented Nov 18, 2023 •
edited
Loading

Solved! Minie requires Heart.

Note to self. Next issue:
java.lang.NoSuchMethodException: com.jme3.bullet.control.RigidBodyControl.getGravity()
at java.base/java.lang.Class.getMethod(Class.java:2227)
at org.openide.nodes.PropertySupport$Reflection.(PropertySupport.java:233)
at com.jme3.gde.core.properties.SceneExplorerProperty.(SceneExplorerProperty.java:98)
[catch] at com.jme3.gde.core.sceneexplorer.nodes.AbstractSceneExplorerNode.makeProperty(AbstractSceneExplorerNode.java:187)
at com.jme3.gde.core.sceneexplorer.nodes.JmeRigidBodyControl.createSheet(JmeRigidBodyControl.java:97)

@neph1
Copy link
Contributor Author
neph1 commented Nov 19, 2023

Thanks to ChatGPT for helping out with the PropertyEditor :)

@neph1 neph1 marked this pull request as ready for review November 19, 2023 10:43
@neph1 neph1 mentioned this pull request Nov 19, 2023
Closed
@tonihele
Copy link
Contributor

Looks good to me!

Blender is still downloaded but not anymore bundled with the SDK? Also is it still in the menu somewhere?

@neph1
Copy link
Contributor Author
neph1 commented Nov 20, 2023

The goal is to purge it completely. I'll see if I have missed it somewhere

@tonihele
Copy link
Contributor

The goal is to purge it completely. I'll see if I have missed it somewhere

Yeah, because I don't see the Blender download URL removed in the PR, nor a menu entry. The latter I'm not sure does it exist only in my memory.. but maybe in the tools menu there was a menu item for Blender.

I recently changed the Blender download URL.. let me see if I can find it...

@tonihele
Copy link
Contributor

https://github.com/jMonkeyEngine/sdk/tree/8b07fc80f53e0c0d3133dd87d71774d7d3f42dc2/nbi/stub/ext/infra/build/products

@neph1 neph1 merged commit ccd02d2 into jMonkeyEngine:master Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Running Physics with Mixamo model without physics yields error
2 participants
@neph1 @tonihele
0