Skip to content

Commit 0b08f4f

Browse files
sdiaomrpropellerspeifeng-unityvidurvij-UnityLaurieCheers-unity
authored
Merge Dev to Main for release 0.4.0 (#241)
* Fixing issue with Demo.cs not working with new URDF Importer co-routine * remove unused import (#163) * update package import urls for tcp connector and urdf importer (#166) * Update urdf_tutorial.md (#168) * Added M prefixes to docs (#167) * Adding Editor version to Ros Integration Readme * Updating package dependencies to proper versions Pick and place tutorial was failing to compile its scripts because the package manifest was pointing to an older version of ROS-TCP-Connector. Updated the manifest and also pulled latest changes for ros_tcp_endpoint. Tested by running DemoScene * Updating Contribution Content (#186) * Contributing content * PR feedback * Adding PR template (#191) * fixing typo in ros-unity service tutorial (#198) * Unity service example (#199) * added example of implementing a UnityService * adding tutorial for unity service * added a couple comments to the example code * typos and code standard fixes * consistency: changing the service name for position_service to pos_srv * updating the tutorial to match the corresponding script * Update the list of new physics features (#204) * add new sections for the released new physics features Co-authored-by: Amanda <[email protected]> * Update to 0.3.0 (#207) * Removing old version tags from git URLs in quick_setup Addressing user issue #209. Our quick_setup guide was still recommending users check out v.0.2.0 of our packages. Removed the reference to a specific package version and instead included a note that users can explicitly declare a package version if desired. Tested by making the change locally in the package manifest. * Adding network troubleshooting doc. (#195) * Adding network troubleshooting doc. * Updating tutorials to the glorious new world (#211) * add changelog (#218) add changelog * Forum links to README, config.yml (#219) * Add linter (#227) * Add linter and pre-commit hooks * Formatting * Remove setting up ruby * Resolve conflicts 2 (#233) * Merging dev branch into main (#78) * Reverting params.yaml and updating code in server_endpoint.md * TcpServer class was renamed * convert all files with dos2unix (#76) * Minor tutorial tweaks, update version numbers Co-authored-by: Devin Miller <[email protected]> Co-authored-by: Peifeng Jing <[email protected]> * Updating instructions for URDF repository (#71) * 1. Adding images for URDF tutorial 2. Adding instructions highlighting Disable Collison tag * Adding link to example of disable collision tag in the niryo_one URDF file * Sarah/readme update (#112) * Adding physics roadmap and minor formatting changes * formatting nits * Calling out feature requests * Update for ROS-TCP-Connector namespace changes * Mpinol/simrealrebased (#107) * Adding sim and real pnp files. * Update with new instructions (#147) * Update tutorial text (#148) * AIRO-405 Adding Yamato boilerplate to Robotics Hub (#144) * Adding testing boilerplate * Adding tests to run after message generation * Tweaks to ensure run works in Bokken image * Enabling audio by default, but disabling it in yamato config * Hotfix: Remove DemoScripts asmdef The DemoScripts.asmdef was a bit of experimental cruft that snuck in via my Yamato changes. This causes Demo.cs to compile as an Editor script, which means it can no longer be instantiated in the scene hierarchy (this breaks the demo). Removing the asmdef puts things back to normal. Tested by running the demo. * Use latest main * Fixing issue with Demo.cs not working with new URDF Importer co-routine * remove unused import (#163) * update package import urls for tcp connector and urdf importer (#166) * Update urdf_tutorial.md (#168) * Added M prefixes to docs (#167) * Adding Editor version to Ros Integration Readme * Updating package dependencies to proper versions Pick and place tutorial was failing to compile its scripts because the package manifest was pointing to an older version of ROS-TCP-Connector. Updated the manifest and also pulled latest changes for ros_tcp_endpoint. Tested by running DemoScene * Updating Contribution Content (#186) * Contributing content * PR feedback * Adding PR template (#191) * Unity service example (#199) * added example of implementing a UnityService * adding tutorial for unity service * added a couple comments to the example code * typos and code standard fixes * consistency: changing the service name for position_service to pos_srv * updating the tutorial to match the corresponding script * Update the list of new physics features (#204) * add new sections for the released new physics features Co-authored-by: Amanda <[email protected]> * Update to 0.3.0 (#207) * Removing old version tags from git URLs in quick_setup Addressing user issue #209. Our quick_setup guide was still recommending users check out v.0.2.0 of our packages. Removed the reference to a specific package version and instead included a note that users can explicitly declare a package version if desired. Tested by making the change locally in the package manifest. * Updating tutorials to the glorious new world (#211) * add changelog (#218) add changelog * Forum links to README, config.yml (#219) * Add linter (#227) * Add linter and pre-commit hooks * Formatting * Remove setting up ruby * Fixing issue with Demo.cs not working with new URDF Importer co-routine (#162) Co-authored-by: Devin Miller <[email protected]> * Readme: New Link (#172) * Fixing issue with Demo.cs not working with new URDF Importer co-routine * remove unused import (#163) * README update * PR feedback * Blog post link update Co-authored-by: Devin Miller <[email protected]> Co-authored-by: peifeng-unity <[email protected]> * Added newsletter link (#175) * remove stale merge * Minor formmating * Fix README and git sub module * Revert unintentional change in urdf_appendix.md * Add back tutorials/pick_and_place/ROS/src/ros_tcp_endpoint Co-authored-by: LaurieCheers-unity <[email protected]> Co-authored-by: Devin Miller <[email protected]> Co-authored-by: Peifeng Jing <[email protected]> Co-authored-by: vidurvij-Unity <[email protected]> Co-authored-by: sarah-gibson <[email protected]> Co-authored-by: Devin Miller (Unity) <[email protected]> Co-authored-by: peifeng-unity <[email protected]> Co-authored-by: Amanda <[email protected]> Co-authored-by: Hamid Younesy <[email protected]> * Update Demo (#235) * Update Demo * Minor fix * Update tutorial manifest to use v0.4.0 * Update ros_tcp_endpoint module * Update quick_setup version * Update tutorial manifest * added chmod to make sure the python scripts are executable (#239) * Minor update in README.md * minor fix Co-authored-by: Devin Miller <[email protected]> Co-authored-by: peifeng-unity <[email protected]> Co-authored-by: vidurvij-Unity <[email protected]> Co-authored-by: LaurieCheers-unity <[email protected]> Co-authored-by: Devin Miller (Unity) <[email protected]> Co-authored-by: Amanda <[email protected]> Co-authored-by: Hamid Younesy <[email protected]> Co-authored-by: Peifeng Jing <[email protected]> Co-authored-by: sarah-gibson <[email protected]>
1 parent b7763a5 commit 0b08f4f

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

50 files changed

+548
-282
lines changed

.editorconfig

+75
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,75 @@
1+
; see http://editorconfig.org/ for docs on this file
2+
; See https://github.com/dotnet/format for dotnet format
3+
4+
root = true
5+
6+
[*]
7+
ignore_if_in_header = This code was generated by a tool|<auto-generated>
8+
indent_style = space
9+
indent_size = 4
10+
; uncomment to help with sharing files across os's (i.e. network share or through local vm)
11+
#end_of_line = lf
12+
; avoid a bom, which causes endless problems with naive text tooling
13+
charset = utf-8
14+
trim_trailing_whitespace = true
15+
insert_final_newline = true
16+
; keeping auto-format enabled helps avoid merge hell for projects without CI-based format validation
17+
#disable_auto_format = true
18+
19+
[*.cs]
20+
; uncomment to enable full formatting of c# files
21+
formatters = generic, uncrustify
22+
23+
[*.asmdef]
24+
scrape_api = true
25+
26+
[**/Tests/**.asmdef]
27+
scrape_api = false
28+
29+
[*.Tests.asmdef]
30+
scrape_api = false
31+
32+
[*.md]
33+
indent_size = 2
34+
; trailing whitespace is unfortunately significant in markdown
35+
trim_trailing_whitespace = false
36+
; uncomment to enable basic formatting of markdown files
37+
#formatters = generic
38+
39+
[{Makefile,makefile}]
40+
; tab characters are part of the Makefile format
41+
indent_style = tab
42+
43+
[*.asmdef]
44+
indent_size = 4
45+
46+
[*.json]
47+
indent_size = 2
48+
49+
[*.{vcproj,bat,cmd,xaml,tt,t4,ttinclude}]
50+
end_of_line = crlf
51+
52+
; this VS-specific stuff is based on experiments to see how VS will modify a file after it has been manually edited.
53+
; the settings are meant to closely match what VS does to minimize unnecessary diffs.
54+
[*.{vcxproj,vcxproj.filters}]
55+
indent_style = space
56+
indent_size = 2
57+
end_of_line = crlf
58+
charset = utf-8-bom
59+
trim_trailing_whitespace = true
60+
insert_final_newline = false
61+
; must be broken out because of 51-char bug (https://github.com/editorconfig/editorconfig-visualstudio/issues/21)
62+
[*.{csproj,pyproj,props,targets}]
63+
indent_style = space
64+
indent_size = 2
65+
end_of_line = crlf
66+
charset = utf-8-bom
67+
trim_trailing_whitespace = true
68+
insert_final_newline = false
69+
[*.{sln,sln.template}]
70+
indent_style = tab
71+
indent_size = 4
72+
end_of_line = crlf
73+
charset = utf-8
74+
trim_trailing_whitespace = true
75+
insert_final_newline = false

.github/ISSUE_TEMPLATE/config.yml

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
blank_issues_enabled: false
2+
contact_links:
3+
- name: Unity Robotics Forum
4+
url: https://forum.unity.com/forums/robotics.623/
5+
about: Discussions and questions about Unity Robotics tools, demos, or integrations.

.github/PULL_REQUEST_TEMPLATE.md

+4-3
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ Provide any relevant links here.
1616

1717
## Testing and Verification
1818

19-
Please describe the tests that you ran to verify your changes. Please also provide instructions, ROS packages, and Unity project files as appropriate so we can reproduce the test environment.
19+
Please describe the tests that you ran to verify your changes. Please also provide instructions, ROS packages, and Unity project files as appropriate so we can reproduce the test environment.
2020

2121
### Test Configuration:
2222
- Unity Version: [e.g. Unity 2020.2.0f1]
@@ -27,8 +27,9 @@ Please describe the tests that you ran to verify your changes. Please also provi
2727
## Checklist
2828
- [ ] Ensured this PR is up-to-date with the `dev` branch
2929
- [ ] Created this PR to target the `dev` branch
30-
- [ ] Followed the style guidelines as described in the [Contribution Guidelines](../CONTRIBUTING.md)
30+
- [ ] Followed the style guidelines as described in the [Contribution Guidelines](https://github.com/Unity-Technologies/Unity-Robotics-Hub/blob/main/CONTRIBUTING.md)
3131
- [ ] Added tests that prove my fix is effective or that my feature works
32+
- [ ] Updated the [Changelog](https://github.com/Unity-Technologies/Unity-Robotics-Hub/blob/dev/CHANGELOG.md) and described changes in the [Unreleased section](https://github.com/Unity-Technologies/Unity-Robotics-Hub/blob/dev/CHANGELOG.md#unreleased)
3233
- [ ] Updated the documentation as appropriate
3334

34-
## Other comments
35+
## Other comments

.github/workflows/pre-commit.yaml

+19
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
name: pre-commit
2+
3+
on:
4+
pull_request:
5+
push:
6+
branches: [dev]
7+
8+
jobs:
9+
pre-commit:
10+
runs-on: ubuntu-latest
11+
steps:
12+
- uses: actions/checkout@v2
13+
- uses: actions/setup-python@v2
14+
with:
15+
python-version: 3.7.x
16+
- uses: actions/setup-dotnet@v1
17+
with:
18+
dotnet-version: '3.1.x'
19+
- uses: pre-commit/[email protected]

.gitignore

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22
.swp
33
.idea
44
.vscode/
5-
tutorials/pick_and_place/PickAndPlaceProject/Packages/packages-lock.json
5+
tutorials/pick_and_place/PickAndPlaceProject/Packages/packages-lock.json

.pre-commit-config.yaml

+32
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
repos:
2+
- repo: https://github.com/pre-commit/pre-commit-hooks
3+
rev: v4.0.1
4+
hooks:
5+
- id: mixed-line-ending
6+
exclude: >
7+
(?x)^(
8+
.*cs.meta|
9+
.*.css|
10+
.*.meta|
11+
.*.mat|
12+
.*.preset|
13+
.*.lighting|
14+
.*.dae
15+
)$
16+
args: [--fix=lf]
17+
18+
- id: trailing-whitespace
19+
name: trailing-whitespace-markdown
20+
types: [markdown]
21+
- id: check-merge-conflict
22+
args: [--assume-in-merge]
23+
- id: check-yaml
24+
# Won't handle the templating in yamato
25+
exclude: \.yamato/.*
26+
27+
28+
- repo: https://github.com/dotnet/format
29+
rev: "7e343070a0355c86f72bdee226b5e19ffcbac931"
30+
hooks:
31+
- id: dotnet-format
32+
args: [--folder, --include]

CHANGELOG.md

+21
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# Changelog
2+
3+
All notable changes to this repository will be documented in this file.
4+
5+
The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
6+
7+
## Unreleased
8+
9+
### Upgrade Notes
10+
11+
### Known Issues
12+
13+
### Added
14+
15+
### Changed
16+
17+
### Deprecated
18+
19+
### Removed
20+
21+
### Fixed

CONTRIBUTING.md

+7-7
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Contribution Guidelines
22

3-
Thank you for your interest in contributing to Unity Robotics! To facilitate your
4-
contributions, we've outlined a brief set of guidelines to ensure that your extensions
3+
Thank you for your interest in contributing to Unity Robotics! To facilitate your
4+
contributions, we've outlined a brief set of guidelines to ensure that your extensions
55
can be easily integrated.
66

77
## Communication
@@ -40,10 +40,10 @@ We run continuous integration on all PRs; all tests must be passing before the P
4040

4141
All Python code should follow the [PEP 8 style guidelines](https://pep8.org/).
4242

43-
All C# code should follow the [Microsoft C# Coding Conventions](https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/inside-a-program/coding-conventions).
44-
Additionally, the [Unity Coding package](https://docs.unity3d.com/Packages/[email protected]/manual/index.html)
45-
can be used to format, encode, and lint your code according to the standard Unity
46-
development conventions. Be aware that these Unity conventions will supersede the
43+
All C# code should follow the [Microsoft C# Coding Conventions](https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/inside-a-program/coding-conventions).
44+
Additionally, the [Unity Coding package](https://docs.unity3d.com/Packages/[email protected]/manual/index.html)
45+
can be used to format, encode, and lint your code according to the standard Unity
46+
development conventions. Be aware that these Unity conventions will supersede the
4747
Microsoft C# Coding Conventions where applicable.
4848

4949
Please note that even if the code you are changing does not adhere to these guidelines,
@@ -60,5 +60,5 @@ email us at [[email protected]](mailto:[email protected]).
6060

6161
## Contribution review
6262

63-
Once you have a change ready following the above ground rules, simply make a
63+
Once you have a change ready following the above ground rules, simply make a
6464
pull request in GitHub.

README.md

+10-10
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,7 @@ A robot simulation demonstrating Unity's new physics solver (no ROS dependency).
3333
| Tutorial | Description |
3434
|---|---|
3535
| [ROS–Unity Integration](tutorials/ros_unity_integration/README.md) | A set of component-level tutorials showing how to set up communication between ROS and Unity |
36-
| [URDF Importer](tutorials/urdf_importer/urdf_tutorial.md) | Steps on using the Unity package for loading [URDF](http://wiki.ros.org/urdf) files |
37-
36+
| [URDF Importer](tutorials/urdf_importer/urdf_tutorial.md) | Steps on using the Unity package for loading [URDF](http://wiki.ros.org/urdf) files |
3837

3938
## Component Repos
4039

@@ -57,7 +56,7 @@ Unite Now 2020: Simulating Robots with ROS and Unity [video](https://resources.u
5756
- (August 26, 2020)
5857
Announcing Unity Robotic Simulation [blog post](https://unity.com/solutions/automotive-transportation-manufacturing/robotics)
5958
- (May 20, 2020)
60-
Use articulation bodies to easily prototype industrial designs with realistic motion and behavior [blog post](https://blogs.unity3d.com/2020/05/20/use-articulation-bodies-to-easily-prototype-industrial-designs-with-realistic-motion-and-behavior/)
59+
Use articulation bodies to easily prototype industrial designs with realistic motion and behavior [blog post](https://blogs.unity3d.com/2020/05/20/use-articulation-bodies-to-easily-prototype-industrial-designs-with-realistic-motion-and-behavior/)
6160

6261
### More from Unity
6362

@@ -85,19 +84,20 @@ Interested in early access to ROS 2 integration? Email [[email protected]
8584

8685
## Community and Feedback
8786

88-
The Unity Robotics projects are open-source and we encourage and welcome contributions.
89-
If you wish to contribute, be sure to review our [contribution guidelines](CONTRIBUTING.md)
87+
The Unity Robotics projects are open-source and we encourage and welcome contributions.
88+
If you wish to contribute, be sure to review our [contribution guidelines](CONTRIBUTING.md)
9089
and [code of conduct](CODE_OF_CONDUCT.md).
9190

9291
## Support
93-
For general questions, feedback, or feature requests, connect directly with the
94-
Robotics team at [[email protected]](mailto:[email protected]).
9592

96-
For bugs or other issues, please file a GitHub issue and the Robotics team will
97-
investigate the issue as soon as possible.
93+
For questions or discussions about Unity Robotics package installations or how to best set up and integrate your robotics projects, please create a new thread on the [Unity Robotics forum](https://forum.unity.com/forums/robotics.623/) and make sure to include as much detail as possible.
9894

99-
## Newsletter
95+
For feature requests, bugs, or other issues, please file a [GitHub issue](https://github.com/Unity-Technologies/Unity-Robotics-Hub/issues) using the provided templates and the Robotics team will investigate as soon as possible.
10096

97+
For any other questions or feedback, connect directly with the
98+
Robotics team at [[email protected]](mailto:[email protected]).
99+
100+
## Newsletter
101101
To get notified about new updates and features, [sign up for our newsletter](https://create.unity3d.com/robotics-simulation-newsletter-sign-up)!
102102

103103
## License

faq.md

+2-11
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,7 @@
77
- [How can I install the Unity Packages without starting from a template project?](#how-can-i-install-the-unity-packages-without-starting-from-a-template-project)
88
- [ROS-TCP Endpoint/Connector](#ros-tcp-endpointconnector)
99
- [How does the TCP Endpoint compare to Rosbridge Server?](#how-does-the-tcp-endpoint-compare-to-rosbridge-server)
10-
- [I receive a `Starting server on ...` message in my ROS terminal, but see a `SocketException: A connection attempt failed because the connected party did not respond after a period of time, or established connection failed because connected host has failed to respond.` in my Unity console.](#i-receive-a-starting-server-on--message-in-my-ros-terminal-but-see-a-socketexception-a-connection-attempt-failed-because-the-connected-party-did-not-respond-after-a-period-of-time-or-established-connection-failed-because-connected-host-has-failed-to-respond-in-my-unity-console)
1110
- [I'm getting the error: `...failed because unknown error handler name 'rosmsg'`.](#im-getting-the-error-failed-because-unknown-error-handler-name-rosmsg)
12-
- [Unity can't connect to ROS!](#unity-cant-connect-to-ros)
1311
- [URDF-Importer](#urdf-importer)
1412
- [I don't see an option to Import Robot from URDF, or I have compile errors upon importing the URDF-Importer.](#i-dont-see-an-option-to-import-robot-from-urdf-or-i-have-compile-errors-upon-importing-the-urdf-importer)
1513
- [Can't find what you're looking for?](#cant-find-what-youre-looking-for)
@@ -23,7 +21,7 @@ How does your Unity integration compare to ROS#?
2321
---
2422
Two of the Unity Robotics repos (URDF Importer and TCP Connector) have been forked from the [Siemens ROS# repo](https://github.com/siemens/ros-sharp).
2523

26-
In the URDF Importer we have added the functionality to instantiate a robot from URDF into a Unity scene with [Articulation Body](https://docs.unity3d.com/2020.2/Documentation/Manual/class-ArticulationBody.html) components on their corresponding joints.
24+
In the URDF Importer we have added the functionality to instantiate a robot from URDF into a Unity scene with [Articulation Body](https://docs.unity3d.com/2020.2/Documentation/Manual/class-ArticulationBody.html) components on their corresponding joints.
2725

2826
Aside from facilitating communication with the TCP Endpoint, the TCP Connector contains the `MessageGeneration` code from ROS#. We added the extra functionality that when generating a C# class from a ROS message, functions are also generated that will serialize and deserialize the messages as ROS would internally.
2927

@@ -58,23 +56,16 @@ The time was logged when the message was sent from Unity before being serialized
5856
- ROS# with ROS Bridge Suite took ~2 seconds
5957
- TCP Connector with TCP Endpoint took ~0.17 seconds
6058

61-
I receive a `Starting server on ...` message in my ROS terminal, but see a `SocketException: A connection attempt failed because the connected party did not respond after a period of time, or established connection failed because connected host has failed to respond.` in my Unity console.
62-
---
63-
This is likely an issue with how your network adapters are set up if you are using a virtual machine. You may need to do some troubleshooting to ensure that your guest OS can talk to your host OS and vice versa. One simple way to do this is to set up a "Host-Only" style network, but this varies based on what virtualization software you're using. Try to ensure that you can `ping` your host OS's IP from inside the guest, and can `ping` the guest from the host, then try this last step again.
64-
6559
I'm getting the error: `...failed because unknown error handler name 'rosmsg'`.
6660
---
6761
This is due to a bug in an outdated package version. Try running `sudo apt-get update && sudo apt-get upgrade` to upgrade.
6862

69-
Unity can't connect to ROS!
70-
---
71-
Ensure that the ROS IP address is entered correctly as the `Host Name` in the RosConnect component in Unity, and if you are using a `params.yaml` file, that the appropriate IP addresses are filled in.
72-
7363
# URDF-Importer
7464

7565
I don't see an option to Import Robot from URDF, or I have compile errors upon importing the URDF-Importer.
7666
---
7767
The [ArticulationBody](https://docs.unity3d.com/2020.2/Documentation/Manual/class-ArticulationBody.html) has dependencies on Unity Editor versions [2020.2.0](https://unity3d.com/unity/whats-new/2020.2.0)+. Try updating your project to the latest 2020.2 release.
7868

69+
7970
# Can't find what you're looking for?
8071
Connect directly with the Robotics team at [[email protected]](mailto:[email protected])!

0 commit comments

Comments
 (0)