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

Karr new sims #15

Merged
merged 15 commits into from
Apr 16, 2024
Merged

Karr new sims #15

merged 15 commits into from
Apr 16, 2024

Conversation

JenniferKarr
Copy link
Contributor

  • Updated recipes.yaml with various calibration and source templates
    single point source, extended galaxy, star field
    darks/detlin for IFU
  • added comments to recipes.yaml to make it easier to read
  • moved definitions of different source types to simulationDefinitions.py
  • added single source, star_field, galaxy sources to simulationDefinitions
  • you can now specify a different input yaml file and output directory when calling run_recipes.py

  --inputYAML: specify input YAML files of recipes
  --outputDir: specify output directory

will default to original values

Moved dictionary / path definitions from start of raw_script.py to an
importable file, to keep user defined input separate from the main script.
 - darks, detlin, flat
 - single sources in imaging, spectra, IFU
 - star field in imaging LM
 - extended galaxy in LM

updated simulationDefinitions.py
  - single sources
  - star field
  - extended galaxy

recipes1.yaml: still in progress templatesx
ESO/testArgs.py Outdated
if(args.inputYAML):
inputYAML = args.inputYAML
else:
inputYAML = "recipes.yaml"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
inputYAML = "recipes.yaml"
from pathlib import Path
inputYAML = Path(__file__).parent / "recipes.yaml"

@hugobuddel
Copy link
Contributor

The github action works again, thanks!

Do you want me to review the code @JenniferKarr ? If so, could you let me know when the PR is ready?

The code seems almost ready, but one of the files is called testArgs.py, so I'm not sure.

Anyway, I think you can also merge it when ready, it's good enough for that.

Deleted extraneous file
@JenniferKarr JenniferKarr merged commit 6cf24bb into main Apr 16, 2024
1 check passed
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