Skip to content

Conversation

@Itay-Tsabary-Starkware
Copy link
Contributor

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware changed the base branch from 11-11-apollo_dashboard_modify_infra_panel_titles to main-v0.14.1 November 13, 2025 09:00
@Itay-Tsabary-Starkware Itay-Tsabary-Starkware force-pushed the 11-13-apollo_dashboard_update_server-side_infra_panels branch from 3cb10ef to a66685b Compare November 13, 2025 09:12
Copy link
Collaborator

@matanl-starkware matanl-starkware left a comment

Choose a reason for hiding this comment

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

@matanl-starkware reviewed 1 of 3 files at r1, 3 of 3 files at r3, all commit messages.
Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @Itay-Tsabary-Starkware)


crates/apollo_dashboard/src/infra_panels.rs line 81 at r3 (raw file):

        let received_msgs_panel = Panel::new(
            "Local request receiving rate",
            format!("Increase of received local requests [{INFRA_INCREASE_DURATION}]"),

Consistency with other panels

Suggestion:

({INFRA_INCREASE_DURATION} window)

crates/apollo_dashboard/src/infra_panels.rs line 85 at r3 (raw file):

            PanelType::TimeSeries,
        );
        let processed_msgs_panel = Panel::new(

Consider using Panel::ratio_time_series for processed/received

Code quote:

Panel::new

crates/apollo_dashboard/src/infra_panels.rs line 112 at r3 (raw file):

            PanelType::TimeSeries,
        );
        let valid_received_msgs_panel = Panel::new(

Consider using Panel::ratio_time_series for valid/total

Code quote:

 Panel::new

crates/apollo_dashboard/src/infra_panels.rs line 118 at r3 (raw file):

            PanelType::TimeSeries,
        );
        let processed_msgs_panel = Panel::new(

Consider using Panel::ratio_time_series for processed/valid

Code quote:

Panel::new

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware force-pushed the 11-13-apollo_dashboard_update_server-side_infra_panels branch from a66685b to ee097e3 Compare November 13, 2025 10:27
Copy link
Contributor Author

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware left a comment

Choose a reason for hiding this comment

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

Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @matanl-starkware)


crates/apollo_dashboard/src/infra_panels.rs line 81 at r3 (raw file):

Previously, matanl-starkware (Matan Lior) wrote…

Consistency with other panels

Done.


crates/apollo_dashboard/src/infra_panels.rs line 85 at r3 (raw file):

Previously, matanl-starkware (Matan Lior) wrote…

Consider using Panel::ratio_time_series for processed/received

What ratio do you suggest displaying here? I want to show the processing rate.


crates/apollo_dashboard/src/infra_panels.rs line 112 at r3 (raw file):

Previously, matanl-starkware (Matan Lior) wrote…

Consider using Panel::ratio_time_series for valid/total

I want rate/increase...


crates/apollo_dashboard/src/infra_panels.rs line 118 at r3 (raw file):

Previously, matanl-starkware (Matan Lior) wrote…

Consider using Panel::ratio_time_series for processed/valid

I want rate/increase...

Copy link
Collaborator

@matanl-starkware matanl-starkware left a comment

Choose a reason for hiding this comment

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

@matanl-starkware reviewed 1 of 2 files at r4, all commit messages.
Reviewable status: 2 of 3 files reviewed, 4 unresolved discussions (waiting on @Itay-Tsabary-Starkware)


crates/apollo_dashboard/src/infra_panels.rs line 85 at r3 (raw file):

Previously, Itay-Tsabary-Starkware wrote…

What ratio do you suggest displaying here? I want to show the processing rate.

ratio increase(processed)/increase(received)
This is what ratio_time_series gives you
(checked it, and the ratio varies around 100% as I would expect).

Copy link
Collaborator

@matanl-starkware matanl-starkware left a comment

Choose a reason for hiding this comment

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

@matanl-starkware reviewed 1 of 2 files at r4.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Itay-Tsabary-Starkware)


crates/apollo_dashboard/src/infra_panels.rs line 85 at r3 (raw file):

Previously, matanl-starkware (Matan Lior) wrote…

ratio increase(processed)/increase(received)
This is what ratio_time_series gives you
(checked it, and the ratio varies around 100% as I would expect).

Too many panels IMO.
Need to reconsider how to arrange the "Infra" rows.

Copy link
Contributor Author

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware left a comment

Choose a reason for hiding this comment

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

@Itay-Tsabary-Starkware reviewed 1 of 3 files at r3, 2 of 2 files at r4, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @matanl-starkware)


crates/apollo_dashboard/src/infra_panels.rs line 85 at r3 (raw file):

Previously, matanl-starkware (Matan Lior) wrote…

ratio increase(processed)/increase(received)
This is what ratio_time_series gives you
(checked it, and the ratio varies around 100% as I would expect).

I want to show the processing rate.

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware added this pull request to the merge queue Nov 13, 2025
Merged via the queue into main-v0.14.1 with commit cb56024 Nov 13, 2025
14 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants