Skip to content

Feat/minimum elev service grid#242

Merged
artistrea merged 10 commits intodevelopmentfrom
feat/minimum_elev_service_grid
Jan 30, 2026
Merged

Feat/minimum elev service grid#242
artistrea merged 10 commits intodevelopmentfrom
feat/minimum_elev_service_grid

Conversation

@brunohcfaria
Copy link

This PR adds a feature to DC-MSS systems where the minimum ground elevation angle of a service grid beam can be set. That parameter limits the field-of-view of satellite beams and prevents very low and distorted beams to be created when the service grid point is far from the servicing satellite.

This PR also adds minor fixes to the S.1528 antenna pattern parameters. The far outside lobe and major_minor_axis_ratio are now input parameters.

@brunohcfaria brunohcfaria self-assigned this Jan 21, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the IMT MSS-DC topology to support a configurable minimum ground elevation angle for service-grid-based beam selection, and exposes additional S.1528 antenna pattern parameters (major/minor axis ratio and far-out side-lobe level) through the configuration system and tests.

Changes:

  • Introduces minimum_service_angle in the service grid parameters, wires it into satellite–grid association logic in TopologyImtMssDc.get_satellite_pointing, and adds tests/YAML defaults for IMT and MSS-D2D profiles.
  • Refactors TopologyImtMssDc.get_coordinates to loop until at least one base station is created, and adds a service-grid-focused test validating that increasing the minimum service angle reduces the number of selected grid points.
  • Adds configurable major_minor_axis_ratio and far_out_side_lobe to S.1528 antenna parameters, uses them in AntennaS1528, and expands tests and test YAML; also tightens validation and error handling in the reference atmosphere model.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
tests/test_topology_imt_mss_dc.py Adds a unit test for minimum_service_angle behavior in service-grid mode and imports calc_elevation to validate the selected grid points vs. the configured elevation threshold.
tests/test_antenna_s1528.py Extends S.1528 antenna tests to set major_minor_axis_ratio and far_out_side_lobe, exercising the new parameterized behavior.
tests/parameters/test_parameters.py Verifies that IMT, MSS-D2D, and single-space-station parameter trees correctly expose minimum_service_angle and S.1528 antenna sub-parameters from the YAML.
tests/parameters/parameters_for_testing.yaml Provides test configuration values for minimum_service_angle in IMT/MSS-D2D and S.1528 major_minor_axis_ratio and far_out_side_lobe under single_space_station.
sharc/topology/topology_imt_mss_dc.py Refactors coordinate generation into an outer loop that repeats until at least one base station is produced, and updates service-grid satellite pointing to compute and filter per-grid-point elevation using minimum_service_angle.
sharc/propagation/atmosphere.py Normalizes the season argument to lowercase and adds validation to reject invalid season names with a ValueError.
sharc/parameters/imt/parameters_grid.py Adds the minimum_service_angle parameter with validation to the service grid configuration (ParametersSatelliteWithServiceGrid).
sharc/parameters/antenna/parameters_antenna_s1528.py Extends S.1528 antenna parameters to include major_minor_axis_ratio and far_out_side_lobe fields used by the antenna model.
sharc/antenna/antenna_s1528.py Updates the S.1528 antenna implementation to accept configurable far_out_side_lobe and major_minor_axis_ratio, adjusting derived parameters (e.g., a, l_b) accordingly and improving error messaging on invalid l_s.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@artistrea artistrea self-requested a review January 23, 2026 02:04
Copy link
Member

@artistrea artistrea left a comment

Choose a reason for hiding this comment

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

Changes should be done as requested by copilot, considering the unresolved comments

brunohcfaria and others added 3 commits January 29, 2026 11:30
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
"visilibity" in the test docstring is misspelled; it should be "visibility" to match the parameter name and improve clarity.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
These two consecutive assertions both check major_minor_axis_ratio against 1.0, which is redundant and makes the test harder to maintain; one of them can be removed without changing test behavior.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@brunohcfaria
Copy link
Author

Changes should be done as requested by copilot, considering the unresolved comments

Changes were applied. Waiting for approval.

Copy link
Member

@artistrea artistrea left a comment

Choose a reason for hiding this comment

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

lgtm

@artistrea artistrea merged commit 5c30a37 into development Jan 30, 2026
2 checks passed
@artistrea artistrea deleted the feat/minimum_elev_service_grid branch January 30, 2026 12:48
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