Skip to content

Batch getMeasurementData into chunks using filters #986

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

Open
wants to merge 25 commits into
base: gsoc25
Choose a base branch
from

Conversation

OmPals
Copy link
Contributor

@OmPals OmPals commented Jun 1, 2025

Description

In this PR, I have solved the large traceroute challenge faced by the traceroute monitor functionality.
After publishing this PR, A user will be able to load any type of measurement without facing any latency.

Detailed list of improvements:

  1. Regardless of measurement size, the traceroute monitor can visualize a measurement result till 1000 probes, last 5 measurement periods.
  2. Existing functionalities also remains constant. ie. On selecting/deselecting probes and destinations:
    - the list of probes and destinations change in the url params
    - the nodes for probes and destinations change in the traceroute graph
    The above functionality enables a use to focus on a specific source-target traceroute.
  3. RTT chart has a time slider. On changing the slider range, the measurement result is either fetched from cache or fetched from server depending upon the availability of data.

Related issue

#981

How Has This Been Tested?

With these measurement ids: 75404443, 75404134, 44852121

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@dpgiakatos dpgiakatos changed the base branch from master to gsoc25 June 3, 2025 02:09
@OmPals OmPals marked this pull request as ready for review June 10, 2025 06:34
@dpgiakatos dpgiakatos self-requested a review June 16, 2025 03:18
Copy link
Member

@dpgiakatos dpgiakatos left a comment

Choose a reason for hiding this comment

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

The review is on Mattermost. Thanks.

@OmPals OmPals requested a review from dpgiakatos June 18, 2025 04:39
@OmPals
Copy link
Contributor Author

OmPals commented Jun 18, 2025

The review is on Mattermost. Thanks.

Updated according to the discussion. :)

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