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.

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

Please Share Sphinx / Gazebo Instructions in Full... #2

Closed
MAVProxyUser opened this issue Jul 13, 2016 · 20 comments
Closed

Please Share Sphinx / Gazebo Instructions in Full... #2

MAVProxyUser opened this issue Jul 13, 2016 · 20 comments

Comments

@MAVProxyUser
Copy link

Please share the Documentation of setup for Sphinx

https://www.youtube.com/watch?v=wcVjTKF_JYI

Where is the documentation?
http://events.linuxfoundation.org/sites/events/files/slides/drone_simulation_with_gazebo_04_06_16.pdf

"Team created in february 2015, first usable release in fall"
Where is it?

"There’ll always be a Next Xmas -> no deadline missed, never"
Expect it in December then?

@ncarrier
Copy link
Contributor

I don't work anymore on this project and sadly, as far as I know, sphinx hasn't been entirely released open source. I published firmwared as an open source project and the gazebo patches have been contributed back to the project, but the gazebo plugins are not public. The drones firmwares for the simulator are not public too.
I really regret this fact. I asked internally if there is a will to publish it outside of Parrot, but there's not much more I can do, sorry.

@ncarrier
Copy link
Contributor

Breaking news, I was just answered that we are going to release it publicly.
But I cannot announce a release date and under which licences the different parts will be released.

@MAVProxyUser
Copy link
Author

Thanks ncarrier! Sorry to hit you via multiple mediums (including email), I was unsure which had a likely chance of a quick response. I appreciate your time.

@ncarrier
Copy link
Contributor

No problem, but please note that my personal address is carrier.nicolas0@gmail.com, you forgot the 0, even if it saved me one notification ^^

@MAVProxyUser
Copy link
Author

Cheers Nicolas! Thanks again for the prompt response. I don't anticipate mailing your personal mail. =]

@MAVProxyUser
Copy link
Author

Now that Disco is near release is there any update on this? I noticed references to Sphinx in the firmware .plf file.

@ncarrier
Copy link
Contributor

ncarrier commented Sep 7, 2016

I only know that the public release in being worked on at the moment, but can't give you no ETA.
Sphinx is mentioned in the plf because the firmwares used in sphinx have as much as possible in common with the original drone firmwares.

@MAVProxyUser
Copy link
Author

Thanks NCarrier...sorry to bother you again with the line of questions, I know your hands are tied. Are there more appropriate venues to ask this question sans bugging you about it?

@ncarrier
Copy link
Contributor

Hello, in fact, my hands aren't really "tied", it's just that there is no more information to share for now.
I'm in direct contact with the sphinx developers and they are working on a public release at the moment, like I told you previously.
The main delicate points are what to share / hide and under which licenses.

As it is frequently the case, the short answer is: "when it's ready".

@MAVProxyUser
Copy link
Author

Understood... and as I stated in the original post.

"Team created in february 2015, first usable release in fall"
...
"There’ll always be a Next Xmas -> no deadline missed, never"

Expect it in December then? =]

Tis always unfortunate when our hopes, expectations and what we say sets expectations for others.

Hopefully you all can sort this out with god speed.

Thanks.

@ncarrier
Copy link
Contributor

Sorry, but both date were for the internal release, not for the public one, because at the moment when I did the presentation, releasing publicly wasn't yet decided.

@MAVProxyUser
Copy link
Author

I read your slide number 7, and I also heard your words, just the same as everyone else. =]
No need to be pedantic on your end. I get it.

"Discussions ongoing for a public release for app developers"
http://events.linuxfoundation.org/sites/events/files/slides/drone_simulation_with_gazebo_04_06_16.pdf

https://youtu.be/wcVjTKF_JYI?t=415
"We are discussing how, when, and for what will we do a public release, targeted to app developers, so if you want to control a swarm of drones, maybe you don't want to test your code in a real situation and break some drones, With our simulator you can instantiate one to three, ten drones, and try your app literally before doing a real one..."

My point is... don't speak up and tease us until you are ready to actually do what you tease us with. =]

Kind of like releasing FirmwareD... it is seemingly pointless to the layperson. "Developed internally and released publicly" (for what use right now?)

https://github.com/Parrot-Developers/firmwared

"Open Source" seems to be a big point for marketing and gaining attention these days. I like to see people have their source "Open" before they open their mouths about it.

@MAVProxyUser
Copy link
Author

MAVProxyUser commented Sep 15, 2016

https://youtu.be/wcVjTKF_JYI?t=1515
"We've added some features [to Gazebo] and all we could give back to the community has been integrated, and the rest has been proposed and is being reviewed (for SEVERAL months now at this point...)

Can you point me to what Sphinx oriented things you shared back with Gazebo project? I see several fixes from:
$ hg log | grep -i parrot | sort | uniq
user: Nicolas Dessart nicolas.dessart@parrot.com
user: Olivier Crave olivier.crave@parrot.com
user: Samuel Lekieffre samuel.lekieffre@parrot.com

Nothing appears to be directly Sphinx related however...

https://bitbucket.org/osrf/gazebo/commits/branch/wind

summary: merge consecutive if conditions
summary: fix style and add comments and break condition
summary: fix comment style
summary: fix style
summary: make sure gzclient is stopped first, before killing gzserver
summary: set extent to EXTENT_NULL in constructor
summary: merge with default
summary: fix previous commits
summary: fix uncaught exceptions
summary: fix uncaught exceptions
summary: fix resource leaks in objects
summary: fix resource leak
summary: fix resource leak
summary: fix resource leak
summary: fix use after free
summary: fix use after free
summary: free memory allocated by strdup
summary: delete copy constructor and assignment operator
summary: fix wrong pimpl copy in copy constructor
summary: fix coverity issues
summary: add battery init voltage tests after a reset
summary: add Battery::ResetVoltage function
summary: fix adiabatic power computation
summary: Close branch user_camera.
summary: Static user camera when following a model
summary: Fixed name generation of test executable
summary: Do not update the center of mass if object is kinematic
summary: Fix animate examples
summary: Merged from default
summary: Merged from default
summary: fix previous commit
summary: merge with default
summary: merge with gazebo7_1
summary: convert direction to radians
summary: enable/disable wind at the model level in gzclient
summary: merge with gazebo7_1
summary: fix code check issues
summary: add message in changelog
summary: update copyright year
summary: fix documentation
summary: add missing include
summary: change copyright year
summary: fix previous commit
summary: merge with default
summary: update wind plugin documentation
summary: new wind demo with 10 spheres attached with ball joints
summary: use field enable_wind when converting an SDF element from a msgs::Link
summary: fix previous commit
summary: fix previous commit
summary: add brief description for each test
summary: fixed previous commit
summary: changed method names
summary: add wind parameters
summary: merged wind_demo with lift_drag_demo
summary: add missing includes
summary: const-correctness
summary: use std::unique_ptr
summary: use gazebo coding style
summary: removed sdf member
summary: Fixed property name
summary: fixed previous commit
summary: merge with default
summary: removed Get prefix and added const for parameters that do not change in functions
summary: added the wind property at the model level
summary: Remove refs to wind distance and magnitude
summary: Remove wind magnitude and direction parameters
summary: Added wind support
summary: do not recalculate the normals when loading a mesh (too slow)
summary: fix actor animation
summary: fix previous commit
summary: fix previous commit
summary: fix previous commit
summary: merge with default
summary: merge with gazebo7_1
summary: update mass_density in the GUI when temperature or pressure change
summary: add missing param[in]
summary: fill atmosphere msg in parent class
summary: move adiabatic power computation in a separated method
summary: fix expected mass density value in test
summary: update mass density after loading SDF parameters
summary: fix test expected value
summary: use the ideal gas law to calculate mass density of the air at sea level
summary: add default values
summary: clarify definition of temperature gradient
summary: use length of gravity vector
summary: add description for adiabaticPower
summary: fixed density of air computation
summary: added reference for pressure and mass density computations
summary: fix adiabatic power computation
summary: use ASSERT_EQ instead of ASSERT_TRUE
summary: remove duplicate branch
summary: check if Atmosphere::Temperature() is 0
summary: fix mass density and pressure computation
summary: removed comment
summary: check if atmosphere model is already registered
summary: add missing \return
summary: use const
summary: removed unneeded overriden methods
summary: update copyright year
summary: added brief description of each test
summary: use default value in methods to avoid duplicates
summary: fix coding style
summary: removed SL suffix from variable and method names
summary: removed Param/SetParam
summary: make dataPtr private and check if elements exist in SDF
summary: remove Reset() calls in destructor
summary: const-correctness
summary: fix documentation
summary: fix includes
summary: use gazebo coding style
summary: removed Get prefix and added const for parameters that do not change in functions
summary: fixed previous commit
summary: merged with default
summary: Added atmosphere support
summary: merge from default
summary: rename to <use_model_frame>
summary: do not create a scene on client
summary: 2016 instead of 2015
summary: added integration test
summary: add missing const
summary: update documentation
summary: move Q_DECLARE_METATYPE in Guiface.cc
summary: fix coding style
summary: use queued connection
summary: added name in track_visual properties
summary: renamed relative to use_model_frame
summary: initialize trackInheritYaw to false
summary: merged with default
summary: Changed name of property section
summary: Removed is_ prefix
summary: Use std::string instead of QString
summary: fix previous commit
summary: fix previous commit
summary: merged with default
summary: Start gazebo in follow mode
summary: Do not track visual if name is default
summary: Fixed error computation when static is set to false
summary: Update coding style and documentation
summary: merged from default
summary: Fixed merge
summary: Reduced scope of variables
summary: Added option to decide whether camera inherits yaw rotation from tracked model
summary: Merged in nkoenig/gazebo-ocrave/user_camera_follow_nate (pull request #3)
summary: Close branch user_camera_follow_nate
summary: Removed follow_model.proto
summary: merged from user_camera_follow_nate
summary: Update comment
summary: Revert changes from pull request #1980
summary: Revert changes from pull request #1980
summary: merged from visuals_louise
summary: merged from default
summary: merged from default
summary: Merged in chapulina/gazebo-ocrave/visuals_louise (pull request #6)
summary: Close branch visuals_louise
summary: Merged with default
summary: Merged in chapulina/gazebo-ocrave/visuals_louise (pull request #5)
summary: Fixed previous merge
summary: Fixed previous merge
summary: Merged with visuals_nate
summary: Merge with default
summary: Merged with upstream
summary: Merged in nkoenig/gazebo-ocrave/visuals_nate (pull request #4)
summary: Merged in nkoenig/gazebo-ocrave/user_camera_follow_nate (pull request #2)
summary: Fixed typo
summary: New entry in Changelog
summary: Set default tracking distance to 8.0
summary: Static camera when following a model
summary: Use ignition math library
summary: Set the pose of a visual
summary: fix indentation for last commit
summary: fix memory corruption
summary: fix indentation for last commit
summary: fix memory corruption
summary: Fixed copyright date
summary: Fixed name generation of test executable
summary: merge
summary: Backed out changeset 4387a6c297ca
summary: Do not update the center of mass if object is kinematic
summary: Added regression test for issue #1694
summary: Fixed index for torques
summary: Fixed GetRelativeLinearAccel and GetWorldLinearAccel with world solver
summary: Do not update the center of mass if object is kinematic
summary: Initialize member values in Battery constructor
summary: Fixed syntax issues and added comments
summary: Added an example for the Battery class
summary: New Battery class description
summary: Removed BatteryPrivate.hh
summary: Handle multiple batteries
summary: Added a Battery class

@MAVProxyUser
Copy link
Author

Have you heard the joke about the Parrot Gazebo simulator called Sphinx?

https://www.youtube.com/watch?v=Yd0fBXwDBmo

Image of me dying waiting

@slekieffre
Copy link

Please take a look at this topic:
http://forum.developer.parrot.com/t/introducing-parrot-simulator-sphinx/5800
That could be of interest to you.

@ncarrier
Copy link
Contributor

ncarrier commented Jun 8, 2017 via email

@idjelic
Copy link

idjelic commented Jun 8, 2017 via email

@MAVProxyUser
Copy link
Author

MAVProxyUser commented Jun 8, 2017

Wow, given all the challenges internally... I am impressed you all pulled this off. hat tip to you.
This is a beautiful thing
http://developer.parrot.com/docs/sphinx/index.html

# echo "deb http://plf.parrot.com/sphinx/binary `lsb_release -cs`/" | sudo tee /etc/apt/sources.list.d/sphinx.list > /dev/null
# sudo apt-key adv --keyserver keyserver.ubuntu.com --recv-keys 508B1AE5
# sudo apt-get update
# sudo apt-get install parrot-sphinx

@Nat-Dev
Copy link

Nat-Dev commented Jun 12, 2019

Hello is this still available? I have a question
I already enter those commands in the link http://developer.parrot.com/docs/sphinx/index.html to my terminal but it’s seem like it doesn’t work, it says “Unable to locate package parrot-sphinx” can anyone help me? Thanks

@ndessart
Copy link
Member

Hello Nat-Dev,

For Parrot-Sphinx related issues, please use the Parrot developer forum:
https://forum.developer.parrot.com/c/sphinx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants