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

QG: illustrate sensitivity to initial conditions #97

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

cgrudz
Copy link
Contributor

@cgrudz cgrudz commented Mar 17, 2022

No description provided.

cgrudz and others added 3 commits March 17, 2022 12:51
This is a rough draft to demonstrate the effects of perturbing initial
conditions on the subsequent solution of the QG system.  More work
to come, to clarify, simplify and add the observation model
@patnr
Copy link
Collaborator

patnr commented Mar 18, 2022

What is this? Purpose (compared to the other one)? Do you want me to look at it now?

@patnr
Copy link
Collaborator

patnr commented Mar 18, 2022

I used the __name__ = __main__ guard to prevent the script from running during the documentation build, since it crashes on the GH servers (no fortran compiled, no sample available, etc). Sorry about the resulting conflicts.

@patnr patnr changed the title TIdying Tidying Mar 18, 2022
@cgrudz
Copy link
Contributor Author

cgrudz commented Mar 18, 2022

Ah, sorry, I forgot that the most recent commit would enter the pull request. The main item with the pull request was just to consolidate the import statements. On the other hand, the new script that I'm working on is to make a demonstration of the divergence of solutions with respect to small perturbations of initial conditions. This is also intended to give a means to generate a free run ensemble for some post-processing, offline analysis. I think I understood the utilities correctly for this, but let me know if the approach is not sensible. Generating an ensemble of two perturbed solutions seemed to take quite a bit of computation time, so I was going to look back to see if there was any way I constructed this inefficiently.

@patnr
Copy link
Collaborator

patnr commented Mar 18, 2022

the new script that I'm working on is to make a demonstration of the divergence of solutions with respect to small perturbations of initial conditions.

That'd be nice. Most of the other mods' demo.py animate a few different ICs so that one can see their divergence & chaos. Not sure how to best do that for QG, since its plots are 2D, and reducing them to a single var is kinda boring. Can you rename this PR for its now actual purpose?

Generating an ensemble of two perturbed solutions seemed to take quite a bit of computation time, so I was going to look back to see if there was any way I constructed this inefficiently.

I haven't looked at the new code yet. But remember that merely importing dapper.mods.QG will cause the initial QG_samples.npz file to be generated, for the purposes of the demo and for sampling ICs. Not sure if that helps. Also note that the step integration function already supports multiprocessing over the ensemble. I don't think there's much to be done to speed it up.

@patnr
Copy link
Collaborator

patnr commented Mar 18, 2022

I see you're on Mac now? DS_store files???

@cgrudz
Copy link
Contributor Author

cgrudz commented Mar 18, 2022

Ahahaha I thought this was something to do with the docs... XD. I'm begrudgingly on Mac, the new work device came in, but I spend most of my time on SSH, just thought I'd wrap up the merge locally. Will remove this.

@patnr patnr changed the title Tidying QG: illustrate sensitivity to initial conditions Mar 18, 2022
@patnr
Copy link
Collaborator

patnr commented Mar 18, 2022

BTW, you may once again want to pull master, and (since I updated the linting system) re-install your virtual env. And please make sure to read this

@cgrudz
Copy link
Contributor Author

cgrudz commented Mar 18, 2022

Ah OK, I've been getting error messages with the pre-commit system when making a push about issues with authentication so I was neglecting this -- has this been fixed in the new version?

@patnr
Copy link
Collaborator

patnr commented Mar 18, 2022

Ah OK, I've been getting error messages with the pre-commit system when making a push about issues with authentication so I was neglecting this -- has this been fixed in the new version?

I think it might. Please give it a try (full re-install of venv).

@patnr patnr force-pushed the master branch 3 times, most recently from 4c01011 to 6673d03 Compare June 9, 2022 14:49
@patnr patnr force-pushed the master branch 3 times, most recently from 2023668 to 8159568 Compare January 10, 2023 18:47
@patnr patnr force-pushed the master branch 2 times, most recently from c6602e5 to e1c0751 Compare March 29, 2023 17:06
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

2 participants