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

chore: merge nebula/develop #176

Merged
merged 44 commits into from
Sep 24, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
44 commits
Select commit Hold shift + click to select a range
b8b25a6
fix(hesai_hw_interface): add error handling (#131)
mojomex May 21, 2024
2ec268f
fix(expected): add missing includes (#152)
mojomex May 21, 2024
9e1a52a
chore(pre-commit): autoupdate hooks (#154)
mojomex May 22, 2024
384d3c9
refactor(hesai)!: combine Hesai ROS wrappers into a single node (#127)
mojomex May 22, 2024
855f36a
refactor(robosense)!: combine Robosense ROS wrappers into a single no…
mojomex May 24, 2024
af427ef
refactor(velodyne)!: combine Velodyne ROS wrappers into a single node…
mojomex May 24, 2024
6b74fbc
feat(continental_ars548): combined continental ars548 ros wrappers in…
knzo25 May 30, 2024
6cc19f8
fix: remove git submodules to prevent docs build error (#158)
drwnz Jun 2, 2024
18848ab
docs: move README contents to documentation (#159)
drwnz Jun 2, 2024
175c0e8
fix(docs): add site url
mojomex Jun 2, 2024
e4c413a
docs: add subdir to site URL
drwnz Jun 2, 2024
809e13c
fix(docs): correctly install mkdocs-material correctly
mojomex Jun 3, 2024
65e679e
fix(docs): run action on push
mojomex Jun 3, 2024
3398471
fix(nebula_common): move EIGEN_ALIGN16 attribute to be clang-compatib…
mojomex Jun 5, 2024
ddc7e50
fix: non steady clock and watchdog (#156)
knzo25 Jun 6, 2024
46171bf
feat(nebula_ros): all sensors json schema support (#155)
amadeuszsz Jun 6, 2024
1a82516
feat(cmake): make colcon build fast again (#146)
knzo25 Jun 7, 2024
ef32c85
fix(docs): only one docs workflow now
mojomex Jun 7, 2024
44dea6f
docs: completed doc workflow (artifact upload/deploy instead of push)
mojomex Jun 7, 2024
204eb45
feat(srr520): initial driver (#124)
knzo25 Jun 13, 2024
10725e4
chore: fix pre-commit config and adapt code to make all checks pass (…
mojomex Jun 26, 2024
3812ba3
nit(hesai_driver): correct documentation block
drwnz Jul 2, 2024
84db0ff
fix(schema): handle zeros and floats for default config generation (#…
amadeuszsz Jul 4, 2024
fb9a28d
docs(readme): add boost version requirements (#175)
mojomex Jul 5, 2024
e58d5a7
docs: update sensor models and pointcloud types (#171)
mojomex Jul 5, 2024
04c9a0b
feat: pretty-print sensor configurations (#172)
mojomex Jul 5, 2024
8a78e79
chore(docs): satisfy pre-commit's block quote rules
mojomex Jul 9, 2024
5377d1c
chore(hesai_decoders): implement clang-tidy fixes (mostly style-relat…
mojomex Aug 19, 2024
95c3709
fix(hesai): correctly handle return mode when updating parameters (#185)
mojomex Aug 26, 2024
31a26ce
chore: fix missing imports and compiler errors
mojomex Aug 30, 2024
1df487e
chore(nebula_tests): fix missing dependency
mojomex Sep 2, 2024
456cb1e
fix(hesai): fix supported return modes for Hesai sensors (#188)
mojomex Sep 3, 2024
daa30c6
Merge branch 'main' into develop
mojomex Sep 5, 2024
5ed805d
test(vls128): regenerate VLS128 ground truth due to modification in b…
mojomex Sep 5, 2024
8449a24
chore: ignore `nojekyll` spelling in documentation workflow
mojomex Sep 5, 2024
62a7406
feat(hesai_hw_interface): configure and check `SO_RCVBUF` automatical…
mojomex Sep 6, 2024
7b429cb
chore(nebula_ros/launch): make sensor_models enum choices for the `se…
mojomex Sep 6, 2024
723a853
fix(nebula_launch.py): update the generic launch file to forward all …
mojomex Sep 6, 2024
15c1e51
feat(hesai): multicast support (#187)
mojomex Sep 7, 2024
528b732
feat(nebula_ros): support `launch_hw` parameter in console again to m…
mojomex Sep 9, 2024
16e1489
feat(hesai): software-based precise FoV limit & early publishing (#173)
mojomex Sep 18, 2024
0a96780
Merge branch 'main' into develop
mojomex Sep 24, 2024
ee57dc4
ci(pre-commit): autofix
pre-commit-ci[bot] Sep 24, 2024
4f54c3b
chore: version bump to 0.2.0
mojomex Sep 24, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
3 changes: 3 additions & 0 deletions .clang-format
Original file line number Diff line number Diff line change
Expand Up @@ -45,3 +45,6 @@ IncludeCategories:
- Regex: '".*"'
Priority: 1
CaseSensitive: true
---
Language: Json
BasedOnStyle: llvm
38 changes: 25 additions & 13 deletions .cspell.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,45 +5,57 @@
"words": [
"adctp",
"Adctp",
"applicate",
"AT",
"autosar",
"block_id",
"Bpearl",
"calib",
"centi",
"ddeg",
"DHAVE",
"Difop",
"extrinsics",
"fout",
"gprmc",
"gptp",
"Helios",
"Hesai",
"horiz",
"Idat",
"ipaddr",
"manc",
"memcpy",
"mkdoxy",
"Msop",
"nohup",
"nproc",
"nsec",
"ntoa",
"pandar",
"PANDAR",
"PANDARAT",
"PANDARQT",
"PANDARXT",
"Pdelay",
"Piyush",
"piyushk",
"QT",
"rclcpp",
"schedutil",
"srvs",
"STD_COUT",
"stds",
"struct",
"structs",
"UDP_SEQ",
"usec",
"vccint",
"Vccint",
"Vdat",
"Wbitwise",
"XT",
"XTM",
"DHAVE",
"Bpearl",
"Helios",
"Msop",
"Difop",
"gptp",
"Idat",
"Vdat",
"autosar",
"srvs",
"manc",
"ipaddr",
"ntoa"
"yukkysaito"
]
}
1 change: 0 additions & 1 deletion .github/workflows/build-and-test-differential.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,4 +54,3 @@ jobs:
fail_ci_if_error: false
verbose: true
flags: differential

28 changes: 22 additions & 6 deletions .github/workflows/documentation.yml
Original file line number Diff line number Diff line change
@@ -1,28 +1,44 @@
# cspell: ignore nojekyll
name: documentation
on:
push:
branches:
- develop
- main
pull_request:
types: [closed]
permissions:
contents: write
id-token: write
pages: write
jobs:
deploy:
runs-on: ubuntu-latest
if: github.event.pull_request.merged == true
if: github.event_name == 'push' || github.event.pull_request.merged == true
steps:
- uses: actions/checkout@v3
- uses: actions/cache@v3
- uses: actions/checkout@v4
- uses: actions/cache@v4
with:
key: ${{ github.ref }}
path: .cache
- uses: actions/setup-python@v4
with:
python-version: 3.x
- uses: actions/setup-python@v5
with:
python-version: 3.x
- run: |
sudo apt update && sudo apt install -y doxygen && \
pip install mkdocs-material \
mkdocs \
pip install mkdocs \
mkdocs-material \
Jinja2 \
ruamel.yaml && \
pip install git+https://github.com/JakubAndrysek/mkdoxy
- run: mkdocs gh-deploy --force
- run: mkdocs build --site-dir _site
- run: touch _site/.nojekyll
- run: |
chmod -c -R +rX "_site/" | while read line; do
echo "::warning title=Invalid file permissions automatically fixed::$line"
done
- uses: actions/upload-pages-artifact@v3
- uses: actions/deploy-pages@v4
39 changes: 39 additions & 0 deletions .github/workflows/json-schema-check.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
name: json-schema-check

on:
pull_request:
workflow_dispatch:

jobs:
check-if-relevant-files-changed:
runs-on: ubuntu-latest
outputs:
run-check: ${{ steps.paths_filter.outputs.json_or_yaml }}
steps:
- uses: actions/checkout@v4
- uses: dorny/paths-filter@v3
id: paths_filter
with:
filters: |
json_or_yaml:
- '**/schema/*.schema.json'
- '**/config/**/*.param.yaml'

json-schema-check:
needs: check-if-relevant-files-changed
if: needs.check-if-relevant-files-changed.outputs.run-check == 'true'
runs-on: ubuntu-latest
steps:
- name: Check out repository
uses: actions/checkout@v4

- name: Run json-schema-check
uses: autowarefoundation/autoware-github-actions/json-schema-check@main

no-relevant-changes:
needs: check-if-relevant-files-changed
if: needs.check-if-relevant-files-changed.outputs.run-check == 'false'
runs-on: ubuntu-latest
steps:
- name: Dummy step
run: echo "No relevant changes, passing check"
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -29,3 +29,6 @@ site/

# qcreator stuff
CMakeLists.txt.user

# pre-commit
node_modules/
6 changes: 5 additions & 1 deletion .markdown-link-check.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
{
"aliveStatusCodes": [200, 206, 403],
"aliveStatusCodes": [
200,
206,
403
],
"ignorePatterns": [
{
"pattern": "^http://localhost"
Expand Down
25 changes: 13 additions & 12 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,65 +4,66 @@ ci:

repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.1.0
rev: v4.6.0
hooks:
- id: check-json
- id: check-merge-conflict
- id: check-toml
- id: check-xml
- id: check-yaml
args: [--allow-multiple-documents]
- id: detect-private-key
- id: end-of-file-fixer
- id: mixed-line-ending
- id: trailing-whitespace
args: [--markdown-linebreak-ext=md]

- repo: https://github.com/igorshubovych/markdownlint-cli
rev: v0.30.0
rev: v0.40.0
hooks:
- id: markdownlint
args: [-c, .markdownlint.yaml, --fix]

- repo: https://github.com/pre-commit/mirrors-prettier
rev: v2.5.1
rev: v3.1.0
hooks:
- id: prettier

- repo: https://github.com/adrienverge/yamllint
rev: v1.26.3
rev: v1.35.1
hooks:
- id: yamllint

- repo: https://github.com/tier4/pre-commit-hooks-ros
rev: v0.4.0
rev: v0.8.0
hooks:
- id: prettier-package-xml
- id: sort-package-xml

- repo: https://github.com/shellcheck-py/shellcheck-py
rev: v0.8.0.3
rev: v0.10.0.1
hooks:
- id: shellcheck

- repo: https://github.com/scop/pre-commit-shfmt
rev: v3.4.2-1
rev: v3.8.0-1
hooks:
- id: shfmt
args: [-w, -s, -i=4]

- repo: https://github.com/pycqa/isort
rev: 5.12.0
rev: 5.13.2
hooks:
- id: isort

- repo: https://github.com/psf/black
rev: 22.1.0
rev: 24.4.2
hooks:
- id: black
args: [--line-length=100]

- repo: https://github.com/PyCQA/flake8
rev: 4.0.1
rev: 7.0.0
hooks:
- id: flake8
additional_dependencies:
Expand All @@ -78,12 +79,12 @@ repos:
]

- repo: https://github.com/pre-commit/mirrors-clang-format
rev: v13.0.0
rev: v18.1.5
hooks:
- id: clang-format

- repo: https://github.com/cpplint/cpplint
rev: 1.5.5
rev: 1.6.1
hooks:
- id: cpplint
args: [--quiet]
Expand Down
3 changes: 2 additions & 1 deletion .prettierignore
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
*.param.yaml
*.yaml
*.rviz
*.json
9 changes: 6 additions & 3 deletions .yamllint.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
extends: default

ignore: |
*.param.yaml

rules:
braces:
level: error
Expand All @@ -13,10 +10,16 @@ rules:
document-start:
level: error
present: false # Don't need document start markers
ignore:
- .clang-format # Needs '---' between languages
line-length: disable # Delegate to Prettier
truthy:
level: error
check-keys: false # To allow 'on' of GitHub Actions
quoted-strings:
level: error
required: only-when-needed # To keep consistent style
indentation:
spaces: consistent
indent-sequences: true
check-multi-line-strings: false
Loading
Loading