isce-framework / isce-framework/isce3
Incorrect Calculation of peak RCS
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 239
- Forks
- 90
- Avg merge
- 13d 1h
- Merged PRs (30d)
- 5
Description
The calculation of the peak RCS in the file python/packages/isce3/cal/radar_cross_section.py::measure_target_rcs() is missing a scaling factor.
This issue has been resolved at
https://github.com/JensReimann/isce3/tree/peak_RCS_fix
but I'm not permitted to open a pull request for it. Please manually pull the changes.
A more detailed description of the problem can be found in the commit. Comments explaining the rationale for the change have been included in the source code.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in python/packages/isce3/cal/radar_cross_section.py at measure_target_rcs(). Compare the issue's target code with the peak_RCS_fix branch and read the commit description and source comments for the missing scaling factor. Done means the correction is integrated and the peak RCS calculation matches the documented fix.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100