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

Fix writing freeflow speed graph in case of huge speeds #2693

Merged
merged 5 commits into from
May 30, 2020

Conversation

SHaaD94
Copy link
Collaborator

@SHaaD94 SHaaD94 commented May 28, 2020

Removed creation of double array from 0 to max speed for creating the chart and csv.

0.physsimFreeFlowSpeedDistribution.png Before:
image
0.physsimFreeFlowSpeedDistribution.png After:
image

Also added sanity check for max value of speed scaling factor.


This change is Reviewable

@SHaaD94
Copy link
Collaborator Author

SHaaD94 commented May 29, 2020

Test!

1 similar comment
@SHaaD94
Copy link
Collaborator Author

SHaaD94 commented May 29, 2020

Test!

@SHaaD94
Copy link
Collaborator Author

SHaaD94 commented May 29, 2020

Test!

1 similar comment
@SHaaD94
Copy link
Collaborator Author

SHaaD94 commented May 30, 2020

Test!

@REASY REASY merged commit e5226ef into develop May 30, 2020
@REASY REASY deleted the ez/#2620-graph-wrting-fix-4ci branch May 30, 2020 11:28
beam-bot pushed a commit that referenced this pull request Jan 11, 2021
Fix writing freeflow speed graph in case of huge speeds
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