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

Burgers' lspg mlp #25

Open
gokhalen opened this issue Nov 16, 2022 · 4 comments
Open

Burgers' lspg mlp #25

gokhalen opened this issue Nov 16, 2022 · 4 comments

Comments

@gokhalen
Copy link

gokhalen commented Nov 16, 2022

If I modify the Burgers' class so that it exposes the same interface as advection_diffusion1d.py will unsteady_default_lspg_advdiff1d_mlp work out of the box?

The class Burgers1d does not use the same variables as AdvDiff1d, hence the demo crashes.

Is there something else I need to modify/consider?

Thanks,

Nachiket

@fnrizzi
Copy link
Member

fnrizzi commented Nov 16, 2022

if you expose the same interface, then you need to change whatever else was specific to the adv problem.
@jtencer can you please help with this since you are the one who did this? thanks

@gokhalen
Copy link
Author

gokhalen commented Nov 21, 2022

I think I'm getting reasonable results on a Burgers' problem. I've modified the advection diffusion example. However, I have one question: Where does the ROM get information of the boundary conditions? Is it through the u0 attribute of the AdvDiff1d class? The full order model is passed to the runLSPG routine on line 87 here. This information will be useful because I'm trying to reproduce the results of Lee & Carlberg

Edit: I do believe I was able to get how to implement boundary conditions on the ROM. Remark 3.1, section 5.3 and section 6.1 of Lee and Carlberg's paper make it clear. I needed to modify a couple of lines in runLSPG. I will reach out if I need more help.

@fnrizzi
Copy link
Member

fnrizzi commented Nov 21, 2022

the ROM reconstrcuts a FOM state, and the solver enforces BCs on that. the basis already have knowledge of the BCs. does that answer?

@gokhalen
Copy link
Author

I centered the snapshot data and set the fomReferenceState and romState according to the paper and the solutions seem to respect the boundary conditions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants