Skip to content

Conversation

@aburrell
Copy link
Contributor

This pull request:

  1. Breaks up the reusable portions of the default plotting script into functions
  2. Adds tests for the plotting script and component functions

TODO:

  • Test that plotting script works as expected
  • Write unit tests for new functions
  • Write unit test for plotting script

aburrell added 5 commits June 23, 2022 16:16
Added aether header tests for a single file as well as a list of files.
Added a general function for reading data headers and improved the docstrings.
Added a function to load data for plotting.
Added a function to make a standard plot.
Separated elements of the script into useable functions.  Updated the help commands to reflect usage.
@aburrell
Copy link
Contributor Author

@aaronjridley can you run the plotting script for me and see if it makes sense for you?

  1. Switch to this pull request in your repository
  2. If you haven't installed aetherpy using python setup.py develop, run that command from the main directory or install in your testing environment
  3. Move to the aetherpy\aetherpy directory
  4. Run python run_plot_model_results.py and make sure the help commands make sense
  5. Run python run_plot_model_results.py with your desired commands to get the plots or movies you want.
  6. Tell me about the success or failures of these steps

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.

2 participants