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

Task 7 (Trial 4): Rearrange regression analysis and remove duplicate components #58

Open
bhackett1024 opened this issue Dec 18, 2024 · 9 comments

Comments

@bhackett1024
Copy link
Contributor

The regression analysis page needs to be rearranged to remove some duplicate data and fix a problem with the units on timings. On an example this is the text that is currently rendered for an origin (most of this is under an OriginDisplay React component):

Initial Document Load
Timings
Overall
Started at: 2 ms
Elapsed: 4137 ms
Network Round Trips: 20
Timing Details
Detailed Steps
Screenshots
After
Final Screenshot:
Timing Information:
total: 4137 ms ms
network: 3502 ms ms
scheduling: 164 ms ms
mainThread: 368 ms ms
workerThread: 0 ms ms
timer: 0 ms ms
unknown: 103 ms ms
Network Data by Extension:

This is the text and associated components that should be rendered instead:

Initial Document Load
Final Screenshot:
Timing Information:
total: 4137 ms
network: 3502 ms
network round trips: 20
scheduling: 164 ms
mainThread: 368 ms
workerThread: 0 ms
timer: 0 ms
unknown: 103 ms
Network Data by Extension:
@bhackett1024
Copy link
Contributor Author

@openhands-agent fix

2 similar comments
@bhackett1024
Copy link
Contributor Author

@openhands-agent fix

@bhackett1024
Copy link
Contributor Author

@openhands-agent fix

Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

A potential fix has been generated and a draft PR #59 has been created. Please review the changes.

Copy link

The workflow to fix this issue encountered an error. Please check the workflow logs for more information.

Copy link

A potential fix has been generated and a draft PR #60 has been created. Please review the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment