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

Improve rcs #273

Merged
merged 11 commits into from
Mar 26, 2024
Merged

Improve rcs #273

merged 11 commits into from
Mar 26, 2024

Conversation

msz-rai
Copy link
Collaborator

@msz-rai msz-rai commented Mar 25, 2024

No description provided.

Copy link
Contributor

@PawelLiberadzki PawelLiberadzki left a comment

Choose a reason for hiding this comment

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

Rest looks fine, considering it is a pull to feature branch.

test/src/scene/radarTest.cpp Outdated Show resolved Hide resolved
@msz-rai msz-rai merged commit 0479cd0 into feature/radar-power Mar 26, 2024
8 checks passed
@msz-rai msz-rai deleted the feature/radar-rcs-dev branch March 26, 2024 07:25
msz-rai added a commit that referenced this pull request Mar 26, 2024
* Add debug development chnages.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Add fixes for RCS calculation.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Reduce mess in RCS calculation kernel.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Remove not used variable.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Remove not necessary dependency on hit position.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Compute radar energy locally to be independent on lidar pose

* Make update to lower RCS value near 90 and -90 degrees.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Cleanup radar postprocessing node (revert clustering changes)

* Make log constexpr (maybe perf optimization)

* Get rid of M_PIf

* Remove calculateMat function

---------

Signed-off-by: Paweł Liberadzki <[email protected]>
Co-authored-by: Paweł Liberadzki <[email protected]>
msz-rai added a commit that referenced this pull request Mar 26, 2024
* Add power computation

* Fix static_asserts in core.h

* Working prototype, need to fix values

* Make updates to radar power calculation. Make minor fixes.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Add additional radar parameters

* Add noise params and RCS NaN check

* Fix clustering test

* Fix reflecting polarization.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Change not necessary complex to scalar.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Fix ray dir (to reflected dir) for scattered field calculation.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Update azimuth and elevation angles in ray generation in RCS radar test.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Improve rcs (#273)

* Add debug development chnages.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Add fixes for RCS calculation.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Reduce mess in RCS calculation kernel.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Remove not used variable.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Remove not necessary dependency on hit position.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Compute radar energy locally to be independent on lidar pose

* Make update to lower RCS value near 90 and -90 degrees.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Cleanup radar postprocessing node (revert clustering changes)

* Make log constexpr (maybe perf optimization)

* Get rid of M_PIf

* Remove calculateMat function

---------

Signed-off-by: Paweł Liberadzki <[email protected]>
Co-authored-by: Paweł Liberadzki <[email protected]>

* Radar power as power received + noise

* Cleanup parameters

* Remove rcsAngleDistributionTest

* Review changes

---------

Signed-off-by: Paweł Liberadzki <[email protected]>
Co-authored-by: Paweł Liberadzki <[email protected]>
Co-authored-by: Mateusz Szczygielski <[email protected]>
Co-authored-by: Mateusz Szczygielski <[email protected]>
msz-rai added a commit that referenced this pull request Jun 11, 2024
* Add power computation

* Fix static_asserts in core.h

* Working prototype, need to fix values

* Make updates to radar power calculation. Make minor fixes.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Add additional radar parameters

* Add noise params and RCS NaN check

* Fix clustering test

* Fix reflecting polarization.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Change not necessary complex to scalar.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Fix ray dir (to reflected dir) for scattered field calculation.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Update azimuth and elevation angles in ray generation in RCS radar test.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Improve rcs (#273)

* Add debug development chnages.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Add fixes for RCS calculation.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Reduce mess in RCS calculation kernel.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Remove not used variable.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Remove not necessary dependency on hit position.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Compute radar energy locally to be independent on lidar pose

* Make update to lower RCS value near 90 and -90 degrees.

Signed-off-by: Paweł Liberadzki <[email protected]>

* Cleanup radar postprocessing node (revert clustering changes)

* Make log constexpr (maybe perf optimization)

* Get rid of M_PIf

* Remove calculateMat function

---------

Signed-off-by: Paweł Liberadzki <[email protected]>
Co-authored-by: Paweł Liberadzki <[email protected]>

* Radar power as power received + noise

* Cleanup parameters

* Remove rcsAngleDistributionTest

* Review changes

---------

Signed-off-by: Paweł Liberadzki <[email protected]>
Co-authored-by: Paweł Liberadzki <[email protected]>
Co-authored-by: Mateusz Szczygielski <[email protected]>
Co-authored-by: Mateusz Szczygielski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants