Skip to content

New features list.#230

Merged
ralph-lange merged 7 commits into
masterfrom
feature/new_feature_list
Nov 11, 2020
Merged

New features list.#230
ralph-lange merged 7 commits into
masterfrom
feature/new_feature_list

Conversation

@ralph-lange
Copy link
Copy Markdown
Contributor

No description provided.

@ralph-lange ralph-lange marked this pull request as draft November 4, 2020 15:10
@ralph-lange
Copy link
Copy Markdown
Contributor Author

@FranFin, @pablogs9, I completed the list of features from the virtual workshop. I also made a proposal how to group them and how to visualize the grouping (by vertical headlines).

@ralph-lange ralph-lange closed this Nov 4, 2020
@ralph-lange
Copy link
Copy Markdown
Contributor Author

However, I think that the list is too long. I propose to reduce it to seven items by combining always two features, except for 'permissive license' and 'seamless integration with ROS 2' as follows:

  1. Microcontroller-optimized client API supporting all major ROS concepts
  2. Seamless integration with ROS 2
  3. Multi-RTOS support with generic build system
  4. Extremely resource-constrained but flexible middleware
  5. Permissive license
  6. Vibrant community and ecosystem
  7. Long-term maintainability and interoperability

What do you think, @FranFin, @pablogs9?

@ralph-lange ralph-lange reopened this Nov 4, 2020
@jamoralp
Copy link
Copy Markdown
Contributor

jamoralp commented Nov 5, 2020

It seems that some links are broken here, specifically all related to ros2/tf2_filter repository, as depicted from the failed CI execution.

Comment thread _docs/overview/features/index.md Outdated
@ralph-lange
Copy link
Copy Markdown
Contributor Author

It seems that some links are broken here, specifically all related to ros2/tf2_filter repository, as depicted from the failed CI execution.

This was caused by moving the geometry2 repo is not related to this PR. I just fixed it in 0ef39a5 and will rebase this PR.

@ralph-lange ralph-lange force-pushed the feature/new_feature_list branch from 2093b3c to f43f615 Compare November 6, 2020 11:27
Signed-off-by: Ralph Lange <ralph.lange@de.bosch.com>
Signed-off-by: Ralph Lange <ralph.lange@de.bosch.com>
@ralph-lange ralph-lange force-pushed the feature/new_feature_list branch from f43f615 to 52a4aee Compare November 6, 2020 11:38
Signed-off-by: Ralph Lange <ralph.lange@de.bosch.com>
@ralph-lange
Copy link
Copy Markdown
Contributor Author

@jamoralp, @FranFin: This is how a version with just seven features could look like.

Comment thread _docs/overview/features/index.md Outdated
Comment thread _docs/overview/features/index.md Outdated
Comment thread _docs/overview/features/index.md Outdated
Comment thread _docs/overview/features/index.md Outdated
Comment thread _docs/overview/features/index.md Outdated
Comment thread _docs/overview/features/index_old.md Outdated
Comment thread _docs/overview/features/index_old.md Outdated
Comment thread _docs/overview/features/index_old.md Outdated
Signed-off-by: Ralph Lange <ralph.lange@de.bosch.com>
Signed-off-by: Ralph Lange <ralph.lange@de.bosch.com>
Signed-off-by: Ralph Lange <ralph.lange@de.bosch.com>
@ralph-lange ralph-lange force-pushed the feature/new_feature_list branch from b9003b1 to 4396e6e Compare November 10, 2020 07:51
@ralph-lange ralph-lange marked this pull request as ready for review November 11, 2020 09:00
Signed-off-by: Ralph Lange <ralph.lange@de.bosch.com>
@ralph-lange ralph-lange force-pushed the feature/new_feature_list branch from 6d1c5c0 to 8cec42d Compare November 11, 2020 09:01
@ralph-lange
Copy link
Copy Markdown
Contributor Author

@pablogs9, @FranFin, the feature list is ready for merge from my perspective. I made it collapsible for readability. I removed the graphics on this page as it appeared too distracting to me. Maybe, we can come up with a less distracting image just for visualization.
Please note that I slightly changed the description of the Permissive License feature.

@ralph-lange ralph-lange merged commit 40cd2a6 into master Nov 11, 2020
@ralph-lange ralph-lange deleted the feature/new_feature_list branch November 11, 2020 15:31
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.

4 participants