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 26.89%. Comparing base (a3c7ede) to head (82586a9).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8988      +/-   ##
==========================================
- Coverage   27.37%   26.89%   -0.49%     
==========================================
  Files        1302     1297       -5     
  Lines       95845    95265     -580     
  Branches    39146    38869     -277     
==========================================
- Hits        26238    25620     -618     
- Misses      66925    67022      +97     
+ Partials     2682     2623      -59     

Flag	Coverage Δ		*Carryforward flag
differential	11.16% <ø> (?)
total	26.82% <ø> (-0.56%)	⬇️	Carriedforward from 0d327b2

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