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

Quickgen specsim #57

Merged
merged 4 commits into from
Dec 22, 2015
Merged

Quickgen specsim #57

merged 4 commits into from
Dec 22, 2015

Conversation

dkirkby
Copy link
Member

@dkirkby dkirkby commented Dec 22, 2015

This PR is to merge PR #56 into master and fixes #44.

@dkirkby
Copy link
Member Author

dkirkby commented Dec 22, 2015

@rstaten Can you give me a command-line I can use to test quickgen in your new branch? This didn't work:

desisim% quickgen --nspec 10
Need Fibermap file

Is there an easy way to give it a valid fibermap (on my laptop) ?

@rstaten
Copy link
Contributor

rstaten commented Dec 22, 2015

@dkirkby quickgen --simspec simspec-00000002.fits --fibermap fibermap-00000002.fits --nspec 7

@dkirkby
Copy link
Member Author

dkirkby commented Dec 22, 2015

I don't already have a file called fibermap-00000002.fits, so that fails.

@rstaten
Copy link
Contributor

rstaten commented Dec 22, 2015

@dkirkby
Copy link
Member Author

dkirkby commented Dec 22, 2015

Thanks, that works for me and I am merging now.

dkirkby added a commit that referenced this pull request Dec 22, 2015
@dkirkby dkirkby merged commit e50a33f into master Dec 22, 2015
@dkirkby dkirkby deleted the quickgen_specsim branch December 22, 2015 22:28
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.

Migrate desisim to use specsim instead of quicksim
2 participants