Skip to content
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

95th percentile line chart does not display correctly on the Response Times (ms) graph #2540

Closed
2 tasks done
borget opened this issue Jan 10, 2024 · 3 comments · Fixed by #2544
Closed
2 tasks done
Labels

Comments

@borget
Copy link

borget commented Jan 10, 2024

Prerequisites

Description

The 95th percentile line chart does not correctly display over time,

Every time you do a hard refresh on the Charts screen (http://mylocust.com/?tab=charts)
The 95th line starts getting render from the start of the test, and it does not show the historical values.

Example of refreshing the page to see the 95th line behavior

avg_resp_time

The 95th percentile line shows up correctly going directly to the Locust test report

(http://mylocust.com/stats/report?theme=dark)
Screenshot 2024-01-10 at 11 23 50

Command line

-f my_test.py --autostart --modern-ui

Locustfile contents

class FastUser(FastHttpUser):
    tasks = [FastUserTasks]


class FastUserTasks(TaskSet):
    @tag("Task1")
    @task(1)
    def task1(self):
        """
        I omitted how below variables are generated. Overall, there is some logic that generates protobuf messages
        that are sent as binary over http post action when executing the tasks, I have around 22 tasks that send 
        different protobuf messages.
        """
        self.client.post(
            url,
            name=f"/{message_type}",
            data=message,
            headers={
                "Content-Type": "application/octet-stream"
            }
        )

Python version

3.11

Locust version

2.20.1

Operating system

MacOS

@borget borget added the bug label Jan 10, 2024
@cyberw
Copy link
Collaborator

cyberw commented Jan 10, 2024

@andrewbaldwin44 Can you take a look?

@cyberw
Copy link
Collaborator

cyberw commented Jan 12, 2024

@borget Is it working now for you? (use pip install --pre -U locust to get the most recent build)

@borget
Copy link
Author

borget commented Feb 6, 2024

@borget Is it working now for you? (use pip install --pre -U locust to get the most recent build)

Hi @cyberw ! Just wanted to share that the solution is still going strong after a month and I just upgraded to locust== 2.22.0. Huge thanks to everyone for the support

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants