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

dmd/euler example #12

Merged
merged 13 commits into from
Sep 5, 2023
Merged

dmd/euler example #12

merged 13 commits into from
Sep 5, 2023

Conversation

bumpwy
Copy link
Collaborator

@bumpwy bumpwy commented Aug 29, 2023

Finished the examples/dg_euler.py example.

The example was run with default inputs: python dg_euler.py on 4 cores, corresponding to the following options:
<<<
Options used:
--mesh periodic-square.mesh
--problem 1
--refine_serial 0
--refine_parallel 1
--order 3
--ode_solver 4
--t_final 2.0
--time_step -0.01
--cfl_number 0.3
--visualization False
--visit_datafiles False
--visualization_steps 50
--energy_fraction 0.9999
--rdim -1
--crbf 0.9

Comparison with libROM example shows identical output error numbers. Although, pylibROM appears to be somewhat slower.

pylibROM output
<<<
Relative error of DMD dens at t_final: 2.0 is 0.0003180880
Relative error of DMD x_mom at t_final: 2.0 is 0.0002069844
Relative error of DMD y_mom at t_final: 2.0 is 0.0012537761
Relative error of DMD e at t_final: 2.0 is 0.0001376798
Elapsed time for solving FOM: 4.072901e+01
Elapsed time for training DMD: 9.936306e-01
Elapsed time for predicting DMD: 1.587152e-03

libROM output
<<<
Relative error of DMD density (dens) at t_final: 2 is 0.000318088
Relative error of DMD x-momentum (x_mom) at t_final: 2 is 0.00020698435
Relative error of DMD y-momentum (y_mom) at t_final: 2 is 0.0012537761
Relative error of DMD energy (e) at t_final: 2 is 0.0001376798
Elapsed time for solving FOM: 2.597199e-01 second
Elapsed time for training DMD: 8.866020e-01 second
Elapsed time for predicting DMD: 8.613530e-04 second

@dreamer2368 dreamer2368 added the RFR Ready for review label Aug 31, 2023
@bumpwy
Copy link
Collaborator Author

bumpwy commented Aug 31, 2023

Hi @dreamer2368 I've addressed your comments

@dreamer2368 dreamer2368 merged commit 72c8cdf into main Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants