Skip to content

Exp/oob antenna pattern#235

Draft
brunohcfaria wants to merge 13 commits intodevelopmentfrom
exp/oob_antenna_pattern
Draft

Exp/oob antenna pattern#235
brunohcfaria wants to merge 13 commits intodevelopmentfrom
exp/oob_antenna_pattern

Conversation

@brunohcfaria
Copy link

Added support for out-of-band antenna pattern.

This is useful for adjacent (or co-channel with OOBE) studies where the OOB antenna has a different diagram than the in-band antenna. This case usually happens when the in-band antenna is an array model where the OOB antenna pattern is equal to the single-element pattern of the array.

* It's a simple mask where the user defines the mask steps
across adjacent bands.
* Added a new oob_antenna into StationManager that represents
the oob antenna pattern to be used for oob emission interference
calculations.

* WIP: for now only IMT and MSS_D2D stations has this.
antenna pattern.

* The frequency factor should embedded into the Stepped Mask not
to the antenna.
@brunohcfaria brunohcfaria self-assigned this Nov 26, 2025
Comment on lines 570 to 575
# FIXME: for more than 1 sys
# NOTE: sharc impl already doesn't really work with n_sys > 1
# so more would have to be fixed before this
# assert np.all(adj_loss == adj_loss.flat[0])

tx_oob_s = tx_oob - adj_loss[0, :]
Copy link
Member

Choose a reason for hiding this comment

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

If the code keeps indexing the zero-th index, adj_loss[0, :], the assert np.all(adj_loss == adj_loss.flat[0]) should be maintained to confirm expected behavior is happening

Copy link
Author

Choose a reason for hiding this comment

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

It wasn't intentional... I'll revert it back
BTW, exceptions are better for this purpose.

@brunohcfaria brunohcfaria force-pushed the exp/oob_antenna_pattern branch from 4ada35c to 81ddf81 Compare December 2, 2025 14:42
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