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

fix(autoware_pointcloud_preprocessor): resolve issue with FLT_MAX not declared on Jazzy #8586

Conversation

reinzor
Copy link
Contributor

@reinzor reinzor commented Aug 22, 2024

Fixes compilation error on Jazzy:

error: ‘FLT_MAX’ was not declared in this scope

@github-actions github-actions bot added component:sensing Data acquisition from sensors, drivers, preprocessing. (auto-assigned) tag:require-cuda-build-and-test labels Aug 22, 2024
Copy link

github-actions bot commented Aug 22, 2024

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@reinzor reinzor force-pushed the fix/pointcloud-preprocessor-flt-max-not-defined branch from e5f8d65 to c9d52ef Compare August 22, 2024 08:53
@vividf vividf changed the title fix(pointcloud-preprocessor): FLT_MAX not declared fix(autoware_pointcloud_preprocessor): FLT_MAX not declared Aug 26, 2024
@vividf vividf changed the title fix(autoware_pointcloud_preprocessor): FLT_MAX not declared fix(autoware_pointcloud_preprocessor): resolve issue with FLT_MAX not declared on Jazzy Aug 26, 2024
Copy link
Contributor

@vividf vividf left a comment

Choose a reason for hiding this comment

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

LGTM

@vividf vividf added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Aug 26, 2024
Copy link

codecov bot commented Aug 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 23.92%. Comparing base (d95cfae) to head (105bd3a).
Report is 93 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8586      +/-   ##
==========================================
- Coverage   23.93%   23.92%   -0.01%     
==========================================
  Files        1384     1386       +2     
  Lines      102106   102133      +27     
  Branches    38912    38911       -1     
==========================================
- Hits        24435    24432       -3     
- Misses      75195    75226      +31     
+ Partials     2476     2475       -1     
Flag Coverage Δ *Carryforward flag
differential 19.08% <ø> (?)
total 23.92% <ø> (-0.01%) ⬇️ Carriedforward from d95cfae

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vividf
Copy link
Contributor

vividf commented Aug 27, 2024

@reinzor Please run the pre-commit on your local, thanks!

Fixes compilation error on Jazzy:

error: ‘FLT_MAX’ was not declared in this scope
Signed-off-by: Rein Appeldoorn <rein.appeldoorn@nobleo.nl>
@reinzor reinzor force-pushed the fix/pointcloud-preprocessor-flt-max-not-defined branch from c9d52ef to 105bd3a Compare August 27, 2024 09:27
@reinzor
Copy link
Contributor Author

reinzor commented Aug 27, 2024

@reinzor Please run the pre-commit on your local, thanks!

done

@vividf vividf merged commit e709b10 into autowarefoundation:main Aug 29, 2024
30 of 31 checks passed
a-maumau pushed a commit to a-maumau/autoware.universe that referenced this pull request Sep 2, 2024
… declared on Jazzy (autowarefoundation#8586)

fix(pointcloud-preprocessor): FLT_MAX not declared

Fixes compilation error on Jazzy:

error: ‘FLT_MAX’ was not declared in this scope

Signed-off-by: Rein Appeldoorn <rein.appeldoorn@nobleo.nl>
ktro2828 pushed a commit to ktro2828/autoware.universe that referenced this pull request Sep 18, 2024
… declared on Jazzy (autowarefoundation#8586)

fix(pointcloud-preprocessor): FLT_MAX not declared

Fixes compilation error on Jazzy:

error: ‘FLT_MAX’ was not declared in this scope

Signed-off-by: Rein Appeldoorn <rein.appeldoorn@nobleo.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:sensing Data acquisition from sensors, drivers, preprocessing. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) tag:require-cuda-build-and-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants