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

add disperser holo4_001 #135

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

add disperser holo4_001 #135

wants to merge 3 commits into from

Conversation

sylvielsstfr
Copy link
Collaborator

I have just copied all the disperser holo4_003 files into holo4_001 and added the corresponding new configuration in setup.py
I have tested in a stanalone version that Spectractor was able to recognize a fits file having the tag holo4_001
Nothing more.

I am checking this version at usdf now. I suppose when the modifications will be put in master, the new stuff will be available in Spectractor DM version ?
What is the speed of this update in DM ? (will it comes after w_2023_35 ?)

@sylvielsstfr sylvielsstfr linked an issue Sep 6, 2023 that may be closed by this pull request
@jeremyneveu
Copy link
Collaborator

@mfisherlevine is unavailable this week so the delay to update in Spectractor DM version will be too long for your use case. However, you can clone the DESC Spectractor at USDF, switch to this branch, and edit your user_setups file to point Spectractor package to this library, for instance:
setup -j spectractor -r $HOME/repos/SpectractorDESC

@mfisherlevine
Copy link
Collaborator

Just to say that I'm back now, and free to do the update to the new faster version any time this week. This update should be pretty quick and easy I think.

@jeremyneveu
Copy link
Collaborator

Before merging: needs to add the holo4_001 transmission and ratio 2/1 measurements from optical test bench (at the moment they are just a copy holo4_003).

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.

auxtelruns202309
3 participants