Skip to content

Commit

Permalink
Download notebook data (#199)
Browse files Browse the repository at this point in the history
* Fix paths to spectral model files in temperature response

The path to the model for hybrid abundances and photospheric abundances were switched.

* Fix atol in test_temperature_response_coronal.py

Made atol scaled to temperature response to provide a more stringent test.

* Fix atol in test_temperature_response_hybrid.py

Made atol scaled to temperature response to provide a more stringent test.

* Fix atol in test_temperature_response_photospheric.py

Made atol scaled to temperature response to provide a more stringent test.

* Fix atol in test_temperature_response.py

Made atol scaled to temperature response to provide a more stringent test.

* Fix atol, rtol values in test_temperature_response.py

Use atol and rtol values that should lead to passing all tests.

* Fix atol, rtol values in test_temperature_response_coronal.py

Use atol, rtol values that should lead to passing all tests.

* Fix atol, rtol values in test_temperature_response_hybrid.py

Use atol, rtol values that should lead to passing all tests.

* Fix atol, rtol values in test_temperature_response_photospheric.py

Use atol, rtol values that should lead to passing all tests.

* Another fix to test_temperature_response_hybrid.py

needed to scale atol to max value of temperature response

* Another fix to test_temperature_response_coronal.py

need to scale atol to the max value of temperature response

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Created new directory, image_correction, and moved xrt_remove_lightleak and xrt_deconvolve to that directory. Also moved associated tests and data

* small changes to image_correction/__init__.py

* Modified Deconvolving_XRT_images notebook so that instead of using package data, the data is downloaded from the VSO

* added lxml to requirements/docs.txt

* Update docs/notebooks/data_analysis/Deconvolving_XRT_images.ipynb

Co-authored-by: Nick Murphy <[email protected]>

* added zeep to requirements/docs.txt

* Update requirements/docs.txt

Co-authored-by: Nick Murphy <[email protected]>

* Update requirements/docs.txt

Co-authored-by: Nick Murphy <[email protected]>

* Update requirements/docs.txt

Co-authored-by: Nick Murphy <[email protected]>

* pre-compile Deconvolving_XRT_images notebook

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Jonathan Slavin <[email protected]>
Co-authored-by: Joy <[email protected]>
Co-authored-by: Nick Murphy <[email protected]>
  • Loading branch information
5 people authored Oct 27, 2023
1 parent b655e46 commit 691cf6c
Show file tree
Hide file tree
Showing 2 changed files with 41 additions and 12 deletions.
52 changes: 40 additions & 12 deletions docs/notebooks/data_analysis/Deconvolving_XRT_images.ipynb

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions requirements/docs.txt
Original file line number Diff line number Diff line change
Expand Up @@ -21,4 +21,5 @@ sphinx-issues >= 3.0
sphinx_automodapi
sphinx_rtd_theme >= 1.2
sphinxcontrib-bibtex >= 2.5
sunpy[all]
towncrier >= 22.12

0 comments on commit 691cf6c

Please sign in to comment.