Skip to content

[RomApp] Use NumpyOutputProcess in RomManager rather than RomBasisOutputProcess - #14828

Open
Rbravo555 wants to merge 2 commits into
masterfrom
rom/rom_manager_use_numpy_output_no_basis_process
Open

Rbravo555 wants to merge 2 commits into
masterfrom
rom/rom_manager_use_numpy_output_no_basis_process

Conversation

@Rbravo555

@Rbravo555 Rbravo555 commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

📝 Description

For a while, the RomManager contained the following comment:

# FIXME:
# - Use a method (upcoming) for smoothly retrieving solutions. In here we are using the RomBasisOutput process in order to store the solutions

This PR introduces direct snapshot handling inside RomManager, eliminating the reliance on RomBasisOutputProcess and avoiding holding snapshots of a single simulation in RAM unnecessarily.

Note: Future PRs will address the SVD computation for full scalability and usability. In this PR, although snapshots are flushed every step (or at a user-defined interval), they are still loaded back into RAM to compute the SVD. A future PR will introduce an alternative using partitioned methods or libraries like dask or dislib.


🆕 Changelog

  • Refactor: Stopped using RomBasisOutputProcess to store snapshots in RomManager.
  • Feature: Implemented SVD logic directly within RomManager (mirroring RomBasisOutputProcess).
  • Fix: Modified SetUpSimulationInstance to give precedence to parameters passed by RomManager.

@Rbravo555
Rbravo555 marked this pull request as ready for review October 2, 2026 14:50
@Rbravo555
Rbravo555 requested a review from a team as a code owner October 2, 2026 14:50

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants