-
Notifications
You must be signed in to change notification settings - Fork 41
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
There should not be unit in the vertical axis in the 1D Euler solver plot #281
Comments
@j8xixo12 Could you please help fix it or find someone to? |
I’ll find someone to fix it. Does only one vertical axis means that the 2 vertical axis on the right hand side need to be removed ? |
Yes, no more than 1 vertical axis. Vertical plotting range should not change as the code runs. To help view small value, zooming should be implement in the viewer. |
@yungyuc |
Thanks a lot, @j8xixo12 , let me close this one. Yes, please create another issue for enhancing the navigation widgets by home-grown code. If you file it before the sprint, I will bring it there and find hands. |
The 1D Euler solver uses non-dimensional equations. There should not be units in the vertical axis. There should be only one vertical axis, too.
The text was updated successfully, but these errors were encountered: