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

fiberloss problems #72

Closed
sbailey opened this issue Aug 1, 2017 · 2 comments
Closed

fiberloss problems #72

sbailey opened this issue Aug 1, 2017 · 2 comments

Comments

@sbailey
Copy link
Contributor

sbailey commented Aug 1, 2017

I'm trying to generate a fiberloss table for different galaxy parameters following the instructions at http://specsim.readthedocs.io/en/stable/fiberloss.html

quickspecsim -c desi-elg --save-fiberloss desi-elg-fiberloss generates a fits file with 70 HDUs but apparently not the ecsv file needed to pass back into the config file.

quickfiberloss has a number of the shape parameters that I would like to vary (handy instead of having to edit the yaml file for each variation), but it apparently doesn't have a way to save the results, and that script is currently broken:

[cori08 ~] quickfiberloss -n 100 --disk-fraction 0.5 --bulge-fraction 0.5
Traceback (most recent call last):
  File "/global/common/cori/contrib/desi/desiconda/20170613-1.1.4-spectro/code/specsim/v0.9/bin/quickfiberloss", line 11, in <module>
    load_entry_point('specsim==0.9', 'console_scripts', 'quickfiberloss')()
  File "/global/common/cori/contrib/desi/desiconda/20170613-1.1.4-spectro/code/specsim/v0.9/lib/python3.5/site-packages/specsim-0.9-py3.5.egg/specsim/quickfiberloss.py", line 66, in main
    simulator.atmosphere.seeing['fwhm_ref'] = args.seeing * u.arcsec
AttributeError: 'Atmosphere' object has no attribute 'seeing'

I can probably derive what I need from the underlying code, but it would be helpful to have these outer wrapper scripts working again too.

@tolgayapici
Copy link

I committed a fix to the dithering branch. I will request a pull.

@dkirkby
Copy link
Member

dkirkby commented Jul 26, 2018

The command mentioned above is working in the latest tag (v0.12):

quickfiberloss -n 100 --disk-fraction 0.5 --bulge-fraction 0.5

Closing now, but re-open if I misunderstood the issue.

@dkirkby dkirkby closed this as completed Jul 26, 2018
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

No branches or pull requests

3 participants