search

Jacob-Stevens-Haas / gen-experiments

Pysindy experiments. Composable and extensible
MIT License
1 stars 2 forks source link

Add PDE experiment #5

Closed yb6599 closed 1 year ago

Jacob-Stevens-Haas commented 1 year ago

To recap and expand on what I just said in person:

  1. Remove debug.py and any extra commented out code
  2. Get gen_experiments.pdes.run_pde() working with an example posted here.

At that point I'll be ready to help with the merge conflicts

yb6599 commented 1 year ago

I've created a new branch with all the unimplemented PDEs and this branch only contains "diffuse1D" PDE and it should run using gen_experiments.pdes.run_pde(42, group= "diffuse1D", diff_params=gen_experiments.diff_params["test_axis"], opt_params=gen_experiments.opt_params["test"], feat_params=gen_experiments.feat_params["pde"], )

Jacob-Stevens-Haas commented 1 year ago

You asked this earlier, but you need to add from gen_experiments import pdes to __init__.py. Otherwise I get:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'gen_experiments' has no attribute 'pdes'. Did you mean: 'odes'?
yb6599 commented 1 year ago

Hey Jake, I've got python -m gen_experiments ... working for ODEs and PDEs.

Jacob-Stevens-Haas commented 1 year ago

Great! Write full python -m gen_experiments ... command here so that I can attempt it on my machine.

yb6599 commented 1 year ago

For ODEs, python -m gen_experiments lorenz --seed 23 --param sim_params=test --param diff_params=test --param opt_params=test --param feat_params=test For PDEs, python -m gen_experiments diffuse1D --seed 54 --param diff_params=test_axis --param opt_params=test --param feat_params=pde

Jacob-Stevens-Haas commented 1 year ago

Ok, it looks like the pdes almost works. When I run it, I get a lot more output than running the lorenz example. This is probably a bug in mitosis, but I want to make sure.