refactor(ground_segmentation): rework parameter

[prakash-kannaiah]

Description

---

Implement the ROS Node configuration layout described in Autoware Foundation Discussion #3371 for the ground_segmentation package.

• Remove duplication of parameter declaration between the source code and the parameter file.
• Remove the default value from the source code to ensure all parameter values are passed from the parameter files.
• Use types that match those returned from declare_parameter() to prevent confusion and potential coding errors.
• Move the default parameter file to a standardized location. Use the JSON Schema for that.

Tests performed

---

No.

Pre-review checklist for the PR author

---

The PR author must check the checkboxes below when creating the PR.

• I've confirmed the contribution guideline.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

---

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.
• The PR has been properly tested.
• The PR has been reviewed by the code owners.

Post-review checklist for the PR author

---

The PR author must check the checkboxes below before merging.

• There are no open discussions or they are tracked via tickets.
• The PR is ready for merge.

After all checkboxes are checked, anyone with write access can merge the PR.

[github-actions]

Thank you for contributing to the Autoware project!

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

Please ensure:

• You've checked our contribution guidelines.
• Your PR follows our pull request guidelines.
• All required CI checks pass before marking the PR ready for review.

[YoshiRi]

@prakash-kannaiah
Thanks for your work! but could you solve DCO error with sign-off missing?

Commit sha: 7edd0c9, Author: Prakash Kannaiah, Committer: GitHub; The sign-off is missing.
Commit sha: 59edf47, Author: prakash-kannaiah, Committer: prakash-kannaiah; The sign-off is missing.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.30%. Comparing base (b3c69b4) to head (9cda4ea).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8988      +/-   ##
==========================================
- Coverage   28.30%   28.30%   -0.01%     
==========================================
  Files        1484     1484              
  Lines      111090   111097       +7     
  Branches    43151    43154       +3     
==========================================
  Hits        31441    31441              
- Misses      76631    76638       +7     
  Partials     3018     3018              

Flag	Coverage Δ		*Carryforward flag
differential	12.97% <ø> (?)
differential-cuda	12.97% <ø> (?)
total	28.30% <ø> (ø)		Carriedforward from b3c69b4

*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.

[github-actions]

Documentation URL: https://autowarefoundation.github.io/autoware.universe/pr-8988/
Modified URLs:

• https://autowarefoundation.github.io/autoware.universe/pr-8988/perception/autoware_ground_segmentation/

[badai-nguyen]

@prakash-kannaiah Thanks for your PR, could you also fix for json-schema-check / json-schema-check (pull_request)

perception/autoware_ground_segmentation/schema/ground_segmentation.schema.json ❌ Failed

[stale]

This pull request has been automatically marked as stale because it has not had recent activity.