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

Add Burger's dataset and PINO #256

Merged
merged 9 commits into from
Nov 2, 2023
Merged

Conversation

crwhite14
Copy link
Collaborator

This pull request adds the Burger's Equation dataset and adds PINO (with FDM) for FNO.
Note that it replaces the old PR that only added Burger's Equation.

Changes

  • I changed the Burgers dataset to npz and created a new link using figshare.
  • Added scripts/train_burgers.py
  • Updated neuralop/datasets/burgers.py to load the new dataset, burgers.npy
  • Created neuralop/losses, which contains data_losses.py, equation_losses.py, and meta_losses.py

How to run the Burgers dataset

  • Download the dataset from here, and place it into /data/burgers/
  • cd scripts
  • To run FNO normally (with data loss): python train_burgers.py
  • To run PINO (i.e., FNO with only equation loss): python train_burgers.py --config_name=default --config_file=burgers_pino_config.yaml

Copy link
Collaborator

@dhpitt dhpitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good to me. Changes from the PINO branch's trainer are in #253

@JeanKossaifi
Copy link
Member

Looks great @crwhite14 ! I'd just add a small PINO example so it's executed each time to show how to add equation loss (we can do in another PR) and rename the training script train_pino_burgers so it's clear it has equation loss.

@JeanKossaifi
Copy link
Member

Merging.

@JeanKossaifi JeanKossaifi merged commit e16bea8 into neuraloperator:main Nov 2, 2023
1 check passed
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.

None yet

4 participants