-
Notifications
You must be signed in to change notification settings - Fork 53
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
Final changes for glvis-4.3 #284
Conversation
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.
LGTM
What is the idea with quadrature functions? Interpreting them as L2 elements would be very simple 😉 . I have done that in the past in my codes. |
This has been discussed multiple times, but there are essentially 3 options we have considered:
Option 3 is most general and will be required for "Support for visualization of particle data". I personally like option 1, because it does not introduce variations beyond the original data. That works only on quad/hex meshes though. |
I see, we must reach consensus then what way to go, which might take longer than actually implementing it 😁 . |
We can also support both options 😁 |
I found out there is no smooth vizualization in 2D for vector fields atm. This could be also added, but it would require to differentiate the vector-to-scalar function 🤔 |
Conflicts: CHANGELOG
In the future should PRs be pulled into |
No, the current policy is that PRs should go first in (The intention for the release branches was to be small and short-lived) |
MFEM autotest was giving styling errors, so I run "make style" here but the results may not be fully satisfactory. Please review c697e0b |
We should really add |
Oops, my bad 😅 |
Should we add a test stream for quadrature function visualization? For example quadrature.saved.zip corresponding to GLVis/web#12 (comment)? |
That would be wise 😉 , but we do not have a matching example code for that, but I do not know if it is really a rule to have one... 🤔 |
it is not a hard requirement |
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.
Everything looks good, the quadrature test would be nice if it is added, but it goes to data anyway.
While we have all this |
📆 Target release date:
July 15, 2024August 7, 2024🟡 In Progress:
🟢 DONE:
master
numlock
is on #3004.2
to4.3
<key>A</key>
toA
inREADME.md
? See this issue⚪ After the release:
⚫ Postponed: