-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add a View menu, improve grid plotting #176
Conversation
Updates the gui options, and triggers a replotting of the grid. Some plotting settings are in the Preferences dialog. The View menu and preferences dialog are now kept in sync. Fixed the plot_grid keep_limits functionality, so that the xlim and ylim settings are preserved when replotting.
Uses wall intersection to calculate which cells are outside the wall, and compute a penalty mask that can be used for immersed boundaries. Also improve plotting.
Makes plotting in post-processing easier.
Should update the reference grid, but for now ignore variables
Document new features and output variables.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @bendudson, looks good to me.
One remaining problem - there is (I think) a bug in Qt.py's --convert
feature, which means that on this branch the gui is not working unless PySide2 is installed. The hypnotoad_mainWindow.py
file (generated by pyside2-uic) now contains
from PySide2.QtCore import *
But the --convert
feature only does this:
line = line.replace("from PySide2 import", "from Qt import QtCompat,")
line = line.replace("QtWidgets.QApplication.translate",
"QtCompat.translate")
I guess this should be raised as an issue on Qt.py, but the short-term fix would be to edit hypnotoad_mainWindow.py
to use the Qt.py equivalent of the PySide2 import (whatever that is). Or we could just require PySide2, but that's probably more work than sticking with Qt.py.
Co-authored-by: John Omotani <john.omotani@ukaea.uk>
Co-authored-by: John Omotani <john.omotani@ukaea.uk>
Same version as now used in Github CI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me now!
Updates the gui options, and triggers a replotting of the grid.
Some plotting settings are in the Preferences dialog. The View menu and preferences dialog are now kept in sync.
Fixed the plot_grid keep_limits functionality, so that the xlim and ylim settings are preserved when replotting.
doc/whats-new.md
with a summary of the changes