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.

Sign up for GitHub

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

Jump to bottom

Quicklook to Nightwatch #77

Closed
weaverba137 opened this issue Oct 2, 2019 · 5 comments
Closed

Quicklook to Nightwatch #77

weaverba137 opened this issue Oct 2, 2019 · 5 comments
Assignees
@sbailey
Labels
question

Comments

@weaverba137
Copy link
Member

How similar are Nightwatch outputs to Quicklook? That is, could we get away with renaming the current Quicklook directory and tweaking some of the files? Also, does this transition make #49 moot?
@sbailey
Copy link
Contributor

sbailey commented Oct 3, 2019

The nightwatch outputs are quite different from quicklook (though when we do get the point of documenting nightwatch, the doc/DESI_SPECTRO_REDUX/SPECPROD/exposures/ directory will be a good starting point. And yes, #49 is now moot; I just closed it.

@weaverba137
Copy link
Member Author

OK. Should we remove the existing quicklook directory then?

@weaverba137
Copy link
Member Author

@sbailey , is it OK to remove quicklook entirely as part of the 19.9 updates?

@sbailey
Copy link
Contributor

sbailey commented Oct 21, 2019

Yes

@weaverba137
Copy link
Member Author

Thank you. Removed in 32b1e4a.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sbailey sbailey

Labels
question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sbailey @weaverba137