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

ROM : POD Implementation #5

Open
1 of 2 tasks
emvalbuena opened this issue Feb 17, 2021 · 2 comments
Open
1 of 2 tasks

ROM : POD Implementation #5

emvalbuena opened this issue Feb 17, 2021 · 2 comments
Labels
enhancement New feature or request priority:high Next task(s) to do.

Comments

@emvalbuena
Copy link
Owner

emvalbuena commented Feb 17, 2021

Although in theory one can use the solution snapshots as a basis to project the operators, in practice these might define a basis with a large condition number.
Having it go through the POD, even if it is with the same number of basis vectors as snapshots, will produce an orthonormal basis.

Warning: watch-out for standardisations, this could introduce problems.

Deliverable
Build POD space of solution snapshots.

References

@emvalbuena emvalbuena changed the title POD Implementation ROM : POD Implementation Feb 17, 2021
@emvalbuena emvalbuena added this to the Plain Vanilla ROM milestone Feb 17, 2021
@emvalbuena
Copy link
Owner Author

emvalbuena commented May 13, 2021

For the moment I am using the scipy.linalg.orth.

According to docs, "Construct an orthonormal basis for the range of A using SVD".
I need to check again the POD procedure, I am not sure if it is equivalent to the use of this function.

That being said, the POD is quite butchery-like in terms of numerics, no novelty there to add on my side.

Therefore, I am going to leave it like that for the time being, until I polish out other tools of the ROM, such as the (M)DEIM procedure.

@emvalbuena emvalbuena added enhancement New feature or request priority:low Not necessary to keep pushing here for the moment. labels May 13, 2021
@emvalbuena emvalbuena added priority:high Next task(s) to do. and removed priority:low Not necessary to keep pushing here for the moment. labels Jun 21, 2021
@emvalbuena
Copy link
Owner Author

emvalbuena commented Jun 21, 2021

Therefore, I am going to leave it like that for the time being, until I polish out other tools of the ROM, such as the (M)DEIM procedure.

Now that the DEIM and MDEIM implementations are robust, I should get back to this, because I am having some trouble with the reduction of the stiffness matrix in the ALE formulation, and I would like to know if the selection and the construction of the POD modes has anything to do with it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority:high Next task(s) to do.
Projects
None yet
Development

No branches or pull requests

1 participant