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

Web: Move Robot documentation #5853

Merged
merged 322 commits into from
Feb 28, 2023
Merged

Web: Move Robot documentation #5853

merged 322 commits into from
Feb 28, 2023

Conversation

BenjaminDeleze
Copy link

@BenjaminDeleze BenjaminDeleze commented Feb 7, 2023

Tasks

  • Move all the documentation of the protos to webots.cloud.
  • I also took the opportunity to improve some of the protos
  • I removed the iCub as it was : ugly, not up to date and not a proto
  • Turn the Scout into a proto (was only a world) as the demo is nice.
  • Rename Tiago++ in Tiagopp
  • Remove old ros module for Ure

Adapting the existing doc:

Add the .proto to webots.cloud

  • Add a yaml file in the project root (parent of /protos) with publish: true
  • Add the proto to webots.cloud/proto
  • Test it (fields in the 3d viewer and documentation)

Additional instruction for robot:

  • Copy the guide/robot_name.md file to projects/robots/.../robot_name/protos/docs/

  • With the docs/guide/images/robots/robots_name:

    • Copy the folder to projects/robots/.../robot_name/protos/docs/images
    • Delete the icon, model, and thumbnail.png pictures
  • In the .md file:

    • Remove the title and the first image
    • Change the path of images to remove the robots folder
    • Modify urls that refered to base nodes (robots, solid) or other documentation pages
    • Remove the file location section
  • In the proto file:

    • Change the documentation url in the .proto file (make it point toward released even if it is not valid yet)

Status

  • Appearances
  • Robots
    • projects/robots/a4/portal/protos/Portal.proto
    • projects/robots/abb/irb/protos/Irb4600-40.proto
    • projects/robots/adept/pioneer2/protos/Pioneer2.proto
    • projects/robots/adept/pioneer3/protos/Pioneer3at.proto
    • projects/robots/adept/pioneer3/protos/Pioneer3DistanceSensor.proto
    • projects/robots/adept/pioneer3/protos/Pioneer3dx.proto
    • projects/robots/adept/pioneer3/protos/Pioneer3Gripper.proto
    • projects/robots/bitcraze/crazyflie/protos/Crazyflie.proto
    • projects/robots/bluebotics/shrimp/protos/Shrimp.proto
    • projects/robots/boston_dynamics/atlas/protos/Atlas.proto
    • projects/robots/boston_dynamics/spot/protos/Spot.proto
    • projects/robots/clearpath/moose/protos/Moose.proto
    • projects/robots/clearpath/pr2/protos/Pr2.proto
    • projects/robots/dji/mavic/protos/Mavic2Pro.proto
    • projects/robots/epfl/biorob/protos/GhostDog.proto
    • projects/robots/epfl/biorob/protos/Salamander.proto
    • projects/robots/epfl/biorob/protos/Yamor.proto
    • projects/robots/epfl/lis/protos/Blimp.proto
    • projects/robots/epson/scara_t6/protos/ScaraT6.proto
    • projects/robots/festo/robotino3/protos/Robotino3HokuyoMountingSupport.proto
    • projects/robots/festo/robotino3/protos/Robotino3Platform.proto
    • projects/robots/festo/robotino3/protos/Robotino3.proto
    • projects/robots/festo/robotino3/protos/Robotino3SickMountingSupport.proto
    • projects/robots/festo/robotino3/protos/Robotino3WebcamMountingSupport.proto
    • projects/robots/festo/robotino3/protos/Robotino3Webcam.proto
    • projects/robots/fp_robotics/p-rob3/protos/P-Grip.proto
    • projects/robots/fp_robotics/p-rob3/protos/P-Rob3.proto
    • projects/robots/franka_emika/panda/protos/PandaHand.proto
    • projects/robots/franka_emika/panda/protos/Panda.proto
    • projects/robots/fujitsu/hoap2/protos/Hoap2.proto
    • projects/robots/gctronic/elisa/protos/Elisa3DistanceSensor.proto
    • projects/robots/gctronic/elisa/protos/Elisa3.proto
    • projects/robots/gctronic/e-puck/protos/E-puckDistanceSensor.proto
    • projects/robots/gctronic/e-puck/protos/E-puckGroundSensors.proto
    • projects/robots/gctronic/e-puck/protos/E-puck.proto
    • projects/robots/gctronic/e-puck/protos/Pi-puck.proto
    • projects/robots/husarion/rosbot/protos/Rosbot.proto
    • projects/robots/irobot/create/protos/Create.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsAxle.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsBase.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsBrickAdapter.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsCube.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsCubieBoxWithCrossSlots.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsCubieBoxWithRoundSlots.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsCubieFemaleCube.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsCubieMaleCube.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsCubiePyramid.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsCubieTriangle.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsDistanceSensor.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsFinger.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsGrabber.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsLEDAppearance.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsLightSensor.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsMotor.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsPivot.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsPlasticAppearance.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsTwister.proto
    • projects/robots/kinematics/tinkerbots/protos/TinkerbotsWheel.proto
    • projects/robots/kondo/khr-2hv/protos/Khr2hv.proto
    • projects/robots/kondo/khr-3hv/protos/Khr3hv.proto
    • projects/robots/k-team/hemisson/protos/Hemisson.proto
    • projects/robots/k-team/khepera1/protos/Khepera1.proto
    • projects/robots/k-team/khepera2/protos/Khepera2.proto
    • projects/robots/k-team/khepera3/protos/Khepera3_DistanceSensor.proto
    • projects/robots/k-team/khepera3/protos/Khepera3_Gripper.proto
    • projects/robots/k-team/khepera3/protos/Khepera3_GroundSensor.proto
    • projects/robots/k-team/khepera3/protos/Khepera3.proto
    • projects/robots/k-team/khepera3/protos/Khepera3_USSensor.proto
    • projects/robots/k-team/khepera4/protos/Khepera4InfraredSensor.proto
    • projects/robots/k-team/khepera4/protos/Khepera4.proto
    • projects/robots/k-team/koala/protos/Koala.proto
    • projects/robots/kuka/youbot/protos/KukaBox.proto
    • projects/robots/kuka/youbot/protos/Youbot.proto
    • projects/robots/lego/mindstorms/protos/MindstormsRover.proto
    • projects/robots/micromagic/mantis/protos/Mantis.proto
    • projects/robots/micromagic/scoute/protos/Scout.proto
    • projects/robots/mir/mir100/protos/Mir100.proto
    • projects/robots/mir/mir100/protos/Mir100UltrasonicSensor.proto
    • projects/robots/mobsya/thymio/protos/Thymio2Ball.proto
    • projects/robots/mobsya/thymio/protos/Thymio2Pen.proto
    • projects/robots/mobsya/thymio/protos/Thymio2.proto
    • projects/robots/nasa/protos/Sojourner.proto
    • projects/robots/neuronics/ipr/protos/IprHd6m180.proto
    • projects/robots/neuronics/ipr/protos/IprHd6m90.proto
    • projects/robots/neuronics/ipr/protos/IprHd6ms180.proto
    • projects/robots/neuronics/ipr/protos/IprHd6ms90.proto
    • projects/robots/nex/protos/FireBird6.proto
    • projects/robots/niryo/ned/protos/Ned.proto
    • projects/robots/nvidia/jetbot/protos/JetBot.proto
    • projects/robots/nvidia/jetbot/protos/JetBotRaspberryPiCamera.proto
    • projects/robots/pal_robotics/tiago_base/protos/TiagoBase.proto
    • projects/robots/pal_robotics/tiago_extensions/protos/TiagoBody.proto
    • projects/robots/pal_robotics/tiago_extensions/protos/TiagoFrontArm.proto
    • projects/robots/pal_robotics/tiago_extensions/protos/TiagoGripper.proto
    • projects/robots/pal_robotics/tiago_extensions/protos/TiagoLeftArm.proto
    • projects/robots/pal_robotics/tiago_extensions/protos/TiagoLeftHey5.proto
    • projects/robots/pal_robotics/tiago_extensions/protos/TiagoRightArm.proto
    • projects/robots/pal_robotics/tiago_extensions/protos/TiagoRightHey5.proto
    • projects/robots/pal_robotics/tiago_iron/protos/TiagoIron.proto
    • projects/robots/pal_robotics/tiagopp/protos/Tiagopp.proto
    • projects/robots/pal_robotics/tiago_steel/protos/TiagoSteel.proto
    • projects/robots/pal_robotics/tiago_titanium/protos/TiagoTitanium.proto
    • projects/robots/parallax/boebot/protos/BoeBot.proto
    • projects/robots/picaxe/microbot/protos/Microbot.proto
    • projects/robots/rec/fabtino/protos/Fabtino.proto
    • projects/robots/robotis/bioloid/protos/BioloidDog.proto
    • projects/robots/robotis/darwin-op/protos/Darwin-opHinge2.proto
    • projects/robots/robotis/darwin-op/protos/Darwin-op.proto
    • projects/robots/robotis/darwin-op/protos/RobotisJersey.proto
    • projects/robots/robotis/darwin-op/protos/RobotisOp2Hinge2.proto
    • projects/robots/robotis/darwin-op/protos/RobotisOp2.proto
    • projects/robots/robotis/darwin-op/protos/RobotisOp3.proto
    • projects/robots/robotis/turtlebot/protos/TurtleBot3Burger.proto
    • projects/robots/robotnik/summit_xl_steel/protos/SummitXlSteel.proto
    • projects/robots/saeon/protos/Altino.proto
    • projects/robots/softbank/nao/protos/BallCandy.proto
    • projects/robots/softbank/nao/protos/Cart.proto
    • projects/robots/softbank/nao/protos/Distributor.proto
    • projects/robots/softbank/nao/protos/KeyHole.proto
    • projects/robots/softbank/nao/protos/KeyPot.proto
    • projects/robots/softbank/nao/protos/Key.proto
    • projects/robots/softbank/nao/protos/LollipopCandy.proto
    • projects/robots/softbank/nao/protos/NaoChallengeRoom.proto
    • projects/robots/softbank/nao/protos/NaoJersey.proto
    • projects/robots/softbank/nao/protos/Nao.proto
    • projects/robots/softbank/nao/protos/NaoRoom.proto
    • projects/robots/softbank/nao/protos/Shelves.proto
    • projects/robots/softbank/nao/protos/Tray.proto
    • projects/robots/softbank/nao/protos/VisualArmature.proto
    • projects/robots/sony/aibo/protos/AiboErs7.proto
    • projects/robots/sony/qrio/protos/QRIO.proto
    • projects/robots/sphero/bb8/protos/BB-8.proto
    • projects/robots/surveyor/protos/SurveyorSrv1.proto
    • projects/robots/unimation/puma/protos/Puma560.proto
    • projects/robots/universal_robots/protos/UR10e.proto
    • projects/robots/universal_robots/protos/UR3e.proto
    • projects/robots/universal_robots/protos/UR5e.proto

Benjamin Deleze and others added 30 commits December 7, 2022 08:21
…proto

Merge develop in feature-web-proto
* abstract camera

* buggy

* advance but still buggy

* working frustum

* always enable

* fix scrollbar and improve style

* improvement (linescale + list of cameras)

* enable and disable optional rendering

* all parameters

* display correctly rows

* Update toolbar.css

* style

* cleanup

* Update ProtoCamera.proto

* Update ProtoCamera.proto

* Update resources/web/wwi/nodes/WbAbstractCamera.js

Co-authored-by: ad-daniel <[email protected]>

* Update resources/web/wwi/nodes/WbCamera.js

Co-authored-by: ad-daniel <[email protected]>

* change color

* parent

* split in two methods

* click and co

* rangefinder

* sanitize abstract camera

* Revert "rangefinder"

This reverts commit 8d43b63.

* sanitize

* renable hover correctly

* Revert "Revert "rangefinder""

This reverts commit 83804e5.

* fix near sanitization

* color

* improve style

* correct color

* progress lidar

* progress

* progress

* progress

* progress

* finish lidar

* const

* cleanup lidar and abstract camera

* code done, but not working

* style

* working

* udpates

* lightSensor

* name

* cleanup

* pen

Co-authored-by: ad-daniel <[email protected]>
Co-authored-by: Olivier Michel <[email protected]>
* add delete function in devices

* style
Merge develop in feature-web-proto
* distance sensor class

* fix bugs

* updates

* lookuptable parsing

* add and remove

* improve +/-

* +/- row

* updates (buggy)

* missing init of reset button

* send update

* update lookuptable

* reset and gui

* add scale

* wrenjs url

* add missing tooltip

* fix bool

* fix bulma
* initial implementation (not working)

* initialization

* fixed size

* test on cabinet

* open once

* fix webots:// url handling in local setup

* clean protos

* test.html

* allow more flexibility in urls

* cleanup

* cleanup

* display name
* initial implementation (not working)

* initialization

* fixed size

* test on cabinet

* open once

* fix webots:// url handling in local setup

* clean protos

* test.html

* allow more flexibility in urls

* cleanup

* cleanup

* begin

* display

* update

* 1 2
* implementation

* fix

* even if no parameters
Merge develop in feature-web-proto
* axis and beginning boundingSphere

* bounding sphere in geometry

* progress

* fix bugs and working version

* fix bug elevationgrid

* typo

* first scaling

* convert to device

* fix delete

* classify some solid as device

* show BO of TouchSensor

* cleanup

* relocate viewpoint (buggy)

* cleanup

* fix moveToViewpoint

* title

* motor

* devices details

* rename

* proto coupled motor

* begin mfstring

* solidreference

* css path

* cleanup

* cleanup

* typos

* improve display

* fix for bulma

* fix default position
* Fix infinite dictionary update (#5676)

* Port changes to master branch

* Check if field triggers regeneration before using it after changing the value

* Fix brackets

* Update changelog

* Disable Select button in WbExtendedStringEditor for restricted fields (#5663)

* Update WbExtendedStringEditor.hpp

* Disable select button for restricted fields

* Update changelog-r2023.md

* Fix wrong assertion failure for console redirection (#5671)

* Add null if nothing was read

* Update src/controller/c/robot.c

Co-authored-by: Olivier Michel <[email protected]>

Co-authored-by: Olivier Michel <[email protected]>

* Improve automobile documentation (#5653)

* Add vehicles documentation

* Improve PROTO descriptions and fix default translations

* update icon_creator

* Update images

* Update generated sources

* Update model image

* Add image thumbnails

* Reuse script for generating vehicles doc

* Reuse script for generating vehicles doc

* Fix image path

* Update car.md

* Fix anchors

* Update vehicle-citroen.md

* Update vehicle-lincoln.md

* Update vehicle-range-rover.md

* Update vehicle-tesla.md

* Update vehicle-toyota.md

* Update ToyotaPrius.proto

* Update vehicle-toyota.md

* Update car.md

* Update vehicle-generic.md

* Update BusWheel.proto

* Update vehicle-bmw.md

* Update vehicle-citroen.md

* Update vehicle-toyota.md

* Update BmwX5Wheel.proto

* Update CitroenCZeroFrontWheel.proto

* Update CitroenCZeroRearWheel.proto

* Update ToyotaPrius.proto

* Fix documentation generated from PROTO files

* Fix pep8 error

* Update car.md

* Add car PROTO field descriptions

* Add description to generic vehicle fields

* Regenerate documentation

* Remove duplicated util file

* Add entry in Changelog

* Improve robustness to NaN values in robot window plots (#5680)

* Ignore NaN values when adjusting the plot range

* Add ChangeLog entry

* Better working directory on Windows (#5685)

Co-authored-by: Stefania Pedrazzi <[email protected]>
Co-authored-by: Yannick Goumaz <[email protected]>
Co-authored-by: Olivier Michel <[email protected]>
…web-proto

Merge develop into feature web proto
* init

* Add modal

* List proto nodes

* Load proto-list and parse

* Parse inserted node

* Progress on notification of insertion

* Semi functional node insertion

* cleanup

* add configure button, Allow navigation between sub-proto, Handle NULL nodes

* Progress add-node

* Minor

* Slot filtering

* Filter geometries and appearances

* Fix configure layout

* Add proto icon preview and fix add-node layout

* Change configure button

* Add filter input to layout

* Progress refactoring of NodeSelector

* Finish refactoring

* Restore filtering

* Add accept/cancel buttons, bigger window and cleanup

* Add filtering based on input

* cleanup

* Debug DEF/USE

* Add NULL to node list

* Debug USE/DEF change notifications on webotsJS side

* Remove extra use links

* Fix deletion and insertion propagation

* Functional propagation through USE

* Fix insersion in previously NULL node

* Update button based on selection

* Ensure IS notifications set clones of the origin node

* cleanup

* Highlight selection

* Progress description

* Add description, license, and minor fixes

* Minor

* Fix regeneration

* Debug

* Functional but dirty

* Improve use node notification

* Improve add-node window

* Fix node selector

* Make node selector a separate window and close when clicking outside

* CSS fixes

* Add not found warning

* Remove NULL references

* Add node remover

* Cleanup

* Replace scenetree with root group

* Refactor but broken

* Fix bug

* Commit prior to reset rework

* Rework reset

* Functional reset for sfnode

* Refactor node selector and greatly simplify

* Cleanup

* fix

* add temporary proto list

* cleanup

* Debug partial removal

* Fix deletion rendering bug

* Change css of node button

* Fix back button

* Cleanup node selector

* cleanup x3dscene

* More cleanup

* Add robot descendant check

* Fix checkboxes

* Restore toolbar

* Cleanup

* Remote proto-list

* chrome fixes

* Sort nodes in list

* Use current node as default selection

* Cleanup

* Fix sources

* Fix sources

* Fix loading

* More css fixes

* Fix css again

* Fix progress bar

* Fix progress

* cleanup

* Review fixes

* Fixes

* Debug

* Cleanup

* Debug nested regeneration

* Fix nested proto

* Fix slots

* Fix scenes and animations

* Fix streaming

* Fix margin

* Debug local regeneration

* Cleanup

* Progress

* Fix regeneration

* Cleanup

* Cleanup

* Cleanup

* Finalize

* Missing semicolons

* Review fixes

* Fix vrml writer

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/X3dScene.js

Co-authored-by: Benjamin Délèze <[email protected]>

Co-authored-by: Benjamin Délèze <[email protected]>
* Progress

* Add restriced string insertion

* Add SFFloat restrictions

* Add all other types

* Use selection index to insert

* Handle unsupported types

* Add reset functionality and selection

* Add support for restricted SFNodes

* Fixes

* Cleanup

* Fix refresh for SFNodes

* Fix error

* Finalize

* Review fixes
* refactor and sfvec2

* sfcolor

* fix bug when urls built from js

* MFString

* mfvec2

* mfFloat

* mfint

* MFBool

* add mfRotation

* cleanup

* mfcolor

* fix delete bug

* cleanup

* correct reset

* mf beginning

* Cleanup parameters (#5721)

* remove useless function parameter

* trash position

* clean up trash position

* correct padding of letters

* sfnode

* refactor mf pipeline

* addition cleanup

* furter simplify

* restore css address

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: ad-daniel <[email protected]>

* remove isInt

Co-authored-by: ad-daniel <[email protected]>

Co-authored-by: ad-daniel <[email protected]>
* refactor and sfvec2

* sfcolor

* fix bug when urls built from js

* MFString

* mfvec2

* mfFloat

* mfint

* MFBool

* add mfRotation

* cleanup

* mfcolor

* fix delete bug

* cleanup

* correct reset

* mf beginning

* Cleanup parameters (#5721)

* remove useless function parameter

* trash position

* clean up trash position

* correct padding of letters

* sfnode

* refactor mf pipeline

* addition cleanup

* furter simplify

* restore css address

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: ad-daniel <[email protected]>

* remove isInt

Co-authored-by: ad-daniel <[email protected]>

* Progress

* Progress

* Progress

* Functional insertion and removal

* Fix configure button

* Fix delete row button

* Attempts

* Minor

* Fix deletion

* Funtional MFNode insertion, deletion, changea and refresh

* Fix reset

* Cleanup

* Fixes

* Fix css

* Minor

* Minor

* Minor

* Fix insertion rules

* Fix multiple MF

* Fix list

* Fix alignment

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/NodeSelectorWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/NodeSelectorWindow.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Update resources/web/wwi/protoVisualizer/Vrml.js

Co-authored-by: Benjamin Délèze <[email protected]>

* Fix useless import

Co-authored-by: BenjaminDeleze <[email protected]>
* Sphere for appearances

* remove transformid

* modularize
* Fix grass

* Fix USE-DEF

* Finalize
* Fix transparency

* Fixes
* tabs

* repopulate on MFNode

* update on mf change

* sf

* fix bug and simplify

* improvement

* Update resources/web/wwi/FloatingProtoParameterWindow.js

Co-authored-by: Olivier Michel <[email protected]>

* rename

* fix html

Co-authored-by: Olivier Michel <[email protected]>
Merge Develop in feature-web-proto
* nao md

* nao

* img

* rename

* move folder

* test

* test

* url

* change documentation link

* links

* file location

* add nao license

* insertbefore correct

* appearance

* change

* correct

* remove print

* unified font

* hide useless tabs

* cleanup

* blank

* typo

* cleanup

* change nao url

* additional links

* thumbnail

* grass thumbnail

* remove thumbnail

* omit docs images from test sources

* remove docs images from assets cache

* restore doc

* missing export

* restore reference

* cleanup

* test url

* doc

* yaml

* yaml

* missing export

* update md and for
* finialize prerequisite

* finalize

* missing image
* remove protos and test.html

* Fix devices scale

* solve todos

* Fix devices scale (#5751)

* Fix devices scale

* solve todos

* remove console.log

* modify colormap

* pose to update

* html
@BenjaminDeleze
Copy link
Author

  • The other IPR robots don't appear to link back to IprHd6m90's page
  • In Tiagopp, it should be possible to insert TiagoGripper and TiagoLeftHey5 in endEffectorRightSlot but they don't show up
  • In TiagoRightArm/TiagoLeftArm, I suspect it should be possible to insert the grippers but currently can't
  • In TiagoBase + TiagoBody, it should be possible to insert stuff in the arm slots

It is fixed (the database was just not up to date with the .proto files)

  • Bioloid points to a non-english site, should probably link to http://en.robotis.com/ instead. Also the video is private
  • RobotisOp2 mentions a variant modeled using hinge2, we should probably link to it

It is fixed

  • UR3e links to Robotiq3fGripper but it fails loading

Yes it is normal and it will also be the case for the newly added RobotisOp2Hinge2 link. The links are already written for released as it will eventually be the goal of this PR and it would be a pain to check every proto and md when merging this branch in develop, then in master, then in released.

If you want to see if the link is working, replace released by doc-add-robots in the url or by develop in the case of the Robotiq3fGripper

Copy link
Contributor

@ygoumaz ygoumaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two robots are sinking into the ground:

  • Robotino3SickMountingSupport
  • Moose

In addition, I don't know if it a good idea to fix this in this PR, but the titles of the sample worlds are often missing the link to the actual file on Github. For example, it is correct in the Crazyflie and the ROSbot but not in the Moose.

@BenjaminDeleze
Copy link
Author

Two robots are sinking into the ground:

  • Robotino3SickMountingSupport
  • Moose

I fixed it for the Moose. But I am not sure if it is a good idea to fix i for the Robotino3SickMountingSupport.
Let me explain: the Robotino3SickMountingSupport is a PROTO that is built only to be inserted in the Robotino3 (in the bottomSlot to be exact. In consequence, it is designed to perfectly fit at this position when inserted, without additional translation/rotation.
If I modified the default translation of the Robotino3SickMountingSupport to be well displayed in the PROTO viewer, it will break this. Except if I also modified the position of the bottom slot of the Robotino3 but it would not make a lot of sense.

So I don't see a perfect solution, but I am leaning towards the option of leaving things as they are.

Note: that is not the only PROTO with this problem: for example, the doorKnob, doorLever, cabinetHandle, ...

In addition, I don't know if it a good idea to fix this in this PR, but the titles of the sample worlds are often missing the link to the actual file on Github. For example, it is correct in the Crazyflie and the ROSbot but not in the Moose.

I will do it, as it should not take too long

@BenjaminDeleze
Copy link
Author

I fixed it for the Moose. But I am not sure if it is a good idea to fix i for the Robotino3SickMountingSupport. Let me explain: the Robotino3SickMountingSupport is a PROTO that is built only to be inserted in the Robotino3 (in the bottomSlot to be exact. In consequence, it is designed to perfectly fit at this position when inserted, without additional translation/rotation. If I modified the default translation of the Robotino3SickMountingSupport to be well displayed in the PROTO viewer, it will break this. Except if I also modified the position of the bottom slot of the Robotino3 but it would not make a lot of sense.

So I don't see a perfect solution, but I am leaning towards the option of leaving things as they are.

Note: that is not the only PROTO with this problem: for example, the doorKnob, doorLever, cabinetHandle, ...

The solution we choose after discussion is:

  • if the proto is a slot then ensure that the floor is under the boundingSphere (like we already do for device)
  • if the proto is not a slot, we let it as is.

@BenjaminDeleze
Copy link
Author

n addition, I don't know if it a good idea to fix this in this PR, but the titles of the sample worlds are often missing the link to the actual file on Github. For example, it is correct in the Crazyflie and the ROSbot but not in the Moose.

This is now done.

Copy link
Contributor

@ygoumaz ygoumaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Thank you!

Copy link
Contributor

@ad-daniel ad-daniel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@BenjaminDeleze BenjaminDeleze merged commit e67f35a into develop Feb 28, 2023
@BenjaminDeleze BenjaminDeleze deleted the doc-add-robots branch February 28, 2023 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improve or fix the documentation (MD files only, no software development) enhancement Implementation of a minor feature
Development

Successfully merging this pull request may close these issues.

4 participants