Skip to content

Add MATLAB driver with DEIM#268

Open
nchristensen wants to merge 105 commits intodocfrom
matlab-driver-cleanup
Open

Add MATLAB driver with DEIM#268
nchristensen wants to merge 105 commits intodocfrom
matlab-driver-cleanup

Conversation

@nchristensen
Copy link
Copy Markdown
Collaborator

@nchristensen nchristensen commented May 13, 2025

This pull request adds a MATLAB driver with a DEIM implementation as well as several supporting features:

  • Calculation of the nonlinear convection basis in Fortran
  • Updated, expanded, and restructured documentation
  • Several new examples for DEIM (t2d, shear, ldc)

This pull request is large, but most of it is new or moved files. There are relatively few changes to the existing files.

@kent0
Copy link
Copy Markdown
Collaborator

kent0 commented Feb 3, 2026

@nchristensen, I think setting it to 1 would be safer since I don't know how universally 0-byte memory allocation is supported (or at all).

@nchristensen
Copy link
Copy Markdown
Collaborator Author

Ah, didn't know that wasn't valid Fortran 77. I'll set it to 1 then.

@nchristensen
Copy link
Copy Markdown
Collaborator Author

@Ping-Hsuan, I think you can review the matlab files now. I've moved the old versions of some files to separate directories, that we can delete after the review is complete.

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

Successfully merging this pull request may close these issues.

3 participants