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

Deleted old/legacy files from repository and added some chi-squared recalculation utilities #5

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

Conversation

aringlis
Copy link
Contributor

Deleted the following old/legacy files from the repository:

  • aia_hsi_dem_analysis_manual.pro
  • aia_hsi_dem_analysis_manual_test.pro
  • aia_teem_map.old2.pro
  • aia_teem_map.old3.pro
  • aia_teem_map.old4.pro
  • aia_teem_movie.old1.pro
  • aia_teem_total.old1.pro
  • aia_teem_total.old2.pro
  • aia_teem_total.old3.pro
  • hannnah_aia.pro
  • ospex_script_27_apr_2012.pro
  • get_mask_script.pro

The above files were either old versions of newer routines or test programs for specific cases that are no longer needed.

Added the following new routines:

  • get_combined_chisq.pro
  • get_rhessi_chisq.pro
  • chi_combo.pro

These are utilities to enable re-calculating of chi-squared values easily, without re-doing fits unnecessarily.

Also included:

  • bug fix for aiafile_to_time. The fileset keyword is case-sensitive, so that 'aia' would run successfully but 'AIA' would result in an error.

…s (from AIA and RHESSI) as input, and outputs the combined reduced chi-squared.
…r a given energy range and systematic uncertainty, provided that the original aia_hsi_fit_results.sav file exists.
…m in chi-squared space of a pair of chi-squared maps.
…squared maps as the input or two scalar chi-squared values. Removed chi_combo.pro as functionality now incorporated into get_combined_chisq function.
… file. Also added HSI_IMAGE keyword to test only the region under a RHESSI contour.
@aringlis
Copy link
Contributor Author

Just noticed that this pull request is still open! I'll post an updated description of the changes.

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.

1 participant