Enhanced certification test: now results match analytical solution#94
Enhanced certification test: now results match analytical solution#94
Conversation
mdmitry1
left a comment
There was a problem hiding this comment.
SMLP results are verified by:
- Analytical solution
- Wolfram Alpha
2. Added witness_certify_plot.py to run_certify
mdmitry1
left a comment
There was a problem hiding this comment.
Added certify result visualization
|
Good work! I tried on MacOS the bhn example with certification, have some errors: =========== |
|
On my Ubuntu 22.04 laptop I have another error: File witness_certify.png looks ok. |
|
Before that: should be Which I took into account in above. |
|
Updates fixed the issue on both MacOS and Ubuntu for me. For this or next PR: dump results and data into a results/ data/ directories rather than directly to ./ |
mdmitry1
left a comment
There was a problem hiding this comment.
Merged with master
mdmitry1
left a comment
There was a problem hiding this comment.
All requested changes were implemented
|
@fbrausse : both requested changes are marked as resolved. However, GitHub status is "requested changes". Please, fix |
fbrausse
left a comment
There was a problem hiding this comment.
I've run the BNH example modified here and results and plots look nice.
Added SMLP certify test for BNH problem