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.

Sign up for GitHub

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

Jump to bottom

Integrate Toby's and Syver's code #2

Open
PabRod opened this issue Nov 14, 2023 · 1 comment
Open

Integrate Toby's and Syver's code #2

PabRod opened this issue Nov 14, 2023 · 1 comment
Assignees
@APJansen @SCiarella

Comments

@PabRod
Copy link
Contributor

PabRod commented Nov 14, 2023 •
edited
Loading

We'd like to explore the possibilities of integrating Toby's work on neural closure models on Syver's incompressibleNavierStokes.jl.
@PabRod PabRod added this to Main Nov 14, 2023
@PabRod PabRod converted this from a draft issue Nov 14, 2023
@PabRod PabRod changed the title Figure out how we can integrate Toby's and Syver's code Integrate Toby's and Syver's code Nov 14, 2023
@APJansen
Copy link

Note I have forked Toby's ECNCM_1D to our organisation, and created two branches and corresponding pull requests. This one contains some simple refactorings that make it easier for everyone to play with it, and this one (which branches off of the former) contains a single refactoring I did that is meant to illustrate what I think we should do on a large scale.

@SCiarella SCiarella moved this from Todo to In Progress in Main Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@APJansen APJansen

@SCiarella SCiarella

Labels
None yet
Projects
Main
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
3 participants
@PabRod @APJansen @SCiarella