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

Make the repo work as self-learning ressource #85

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

ct2034
Copy link
Contributor

@ct2034 ct2034 commented Dec 6, 2024

This aims to make this repo work independently of the workshop.

  • mention workshop event not so prominently
  • guide through different readme with prerequisites and next steps sections
  • TBD ...

@ct2034
Copy link
Contributor Author

ct2034 commented Dec 11, 2024

The link to the skiros readme does not work, because it's in a submodule. Do you have any idea how to fix this, @matthias-mayr ? Did you never make a workshop specific readme? Maybe you could write one and put it in the technologies/SkiROS2 folder?

Copy link
Contributor

@sea-bass sea-bass left a comment

Choose a reason for hiding this comment

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

Thanks for taking a pass at this!

I already think there's enough content on my technology (PyRoboSim), so will let others comment on / add to their stuff.

I mostly took an editorial pass.

README.md Outdated Show resolved Hide resolved
problems/README.md Outdated Show resolved Hide resolved
problems/README.md Outdated Show resolved Hide resolved
problems/README.md Outdated Show resolved Hide resolved
problems/README.md Outdated Show resolved Hide resolved
technologies/README.md Outdated Show resolved Hide resolved
technologies/README.md Outdated Show resolved Hide resolved
technologies/README.md Outdated Show resolved Hide resolved
Comment on lines +29 to +39
- [**To start using it to solve the problems, refer to the BehaviorTree.CPP subfolder**](./BehaviorTree.CPP/README.md).

- [ros_bt_py](./ros_bt_py/README.md)
- A Python based library for behavior trees.
- [github repo](https://github.com/fzi-forschungszentrum-informatik/ros2_ros_bt_py)
- [**To start using it to solve the problems, refer to the ros_bt_py subfolder**](./ros_bt_py/README.md).

---
- [FlexBE](./FlexBE/README.md)
- A Python based library for hierarchical finite-state machines.
- [github org](https://github.com/FlexBE)
- [**To start using it to solve the problems, refer to the the FlexBE subfolder**](./FlexBE/README.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

for the last sub-bullets of each, should the URL only be placed on the technology subfolder part, instead of the full sentence?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My thought was to highlight it to differentiate it as a primary navigation path from the other external links. But the bold formatting also does that, to be fair.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My thought was to highlight it to differentiate it from the other (external) links as a primary navigation path. But the bold formatting also does that, to be fair.

ct2034 and others added 9 commits December 17, 2024 09:52
Co-authored-by: Sebastian Castro <[email protected]>
Co-authored-by: Sebastian Castro <[email protected]>
Co-authored-by: Sebastian Castro <[email protected]>
Co-authored-by: Sebastian Castro <[email protected]>
Co-authored-by: Sebastian Castro <[email protected]>
Co-authored-by: Sebastian Castro <[email protected]>
Co-authored-by: Sebastian Castro <[email protected]>
Co-authored-by: Sebastian Castro <[email protected]>
Co-authored-by: Sebastian Castro <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants