Skip to content

Revert "first pass on centralizing math" - #231

Closed
nscheuer wants to merge 1 commit into
mainfrom
revert-227-2_1/math_centralization
Closed

nscheuer wants to merge 1 commit into
mainfrom
revert-227-2_1/math_centralization

Conversation

@nscheuer

Copy link
Copy Markdown
Owner

Reverts #227

@nscheuer nscheuer closed this Oct 15, 2026
@github-actions

Copy link
Copy Markdown

Benchmark Report

Benchmark multiplication factors

Tests Outside Green Range

status category test multiplier
1.38x controllers mtq_w_rw_lp_desat_find_u 1.38x
1.37x controllers mtq_w_rw_find_u 1.37x
1.33x controllers mtq_lovera_find_u 1.33x
1.31x controllers mtq_wisniewski_find_u 1.31x
1.30x controllers mtq_w_rw_qpc_find_u 1.30x
1.28x orbital_state propagate_jacobians_rk4_j2 1.28x
1.27x orbit get_range_sampled_dt 1.27x
1.26x dynamics dynamics_for_solver_full_stack 1.26x
1.25x dynamics dynamics_core_mtq 1.25x
1.24x orbit geocentric_to_ecef_orbit 1.24x
1.24x goals eci_goal_error 1.24x
1.23x controllers mtq_w_rw_qpw_find_u 1.23x
1.23x satellite act_torque_full_stack 1.23x
1.22x satellite sensor_readings_full_stack 1.22x
1.22x disturbances gg_torque_qjac 1.22x
1.22x dynamics noiseless_rk4_full_stack 1.22x
1.21x controllers mtq_w_rw_qpg_find_u 1.21x
1.20x goals coordinate_goal_error 1.20x
1.20x disturbances srp_torque_qjac 1.20x
0.10x goals environment_vector_goal_group 0.10x
0.13x orbital_state propagate_orbit_rk4_full_state 0.13x
0.14x orbit get_os_interpolated 0.14x
0.15x orbit new_orbit_from_times_small 0.15x
0.23x orbit construct_propagated_orbit_small 0.23x

Color bands: green 0.8x-1.2x, yellow 0.5x-0.8x or 1.2x-1.5x, red outside that range.

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.

1 participant