-
Notifications
You must be signed in to change notification settings - Fork 8
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
An remove duplicates interface3 #69
Conversation
AndreaNeumayr
commented
Jan 25, 2022
- remove all duplicates from ModiaInterface in Modia
- using Modia instead of Modia3D, ModiaLang and Modia3D.ModiaInterface
@@ -49,7 +49,7 @@ stopTime = 10.0 | |||
# requiredFinalStates = [-1.578178283450938, 0.061515170100766486] | |||
|
|||
Modia3D.disableRenderer() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better use Scene option 'enableVisualization'.
@@ -2,6 +2,7 @@ module YouBotGripping | |||
|
|||
using ModiaLang | |||
import Modia3D |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is Modia3D still required?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, in this case it is needed for joinpath(Modia3D.path, "objects/robot_KUKA_YouBot/base_frame.obj")
@@ -10,8 +11,6 @@ include("$(ModiaLang.path)/models/Electric.jl") | |||
include("$(ModiaLang.path)/models/Rotational.jl") | |||
include("$(ModiaLang.path)/models/Translational.jl") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In some cases it was possilbe to replace ModiaLang
as well, but here it is needed to include some models.