Skip to content

MMT BINOSPEC SLITMASK DESIGN #2053

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
Open
debora-pe wants to merge 37 commits into
base: develop
Choose a base branch
from
Open

MMT BINOSPEC SLITMASK DESIGN #2053

debora-pe wants to merge 37 commits into from
+716 −4

Conversation

@debora-pe
Copy link
Collaborator

@debora-pe debora-pe commented Jan 14, 2026 •
edited
Loading

This PR primarily add the possibility to ingest in the reduction the information from the MMT Binospec slitmask design.

Additionally, support for 2 new spectrographs is added: magellan_ldss3, magellan_imacs.

feigewang and others added 30 commits November 28, 2020 20:32
@feigewang
LBT MODS1B
b892947
@feigewang
Magellan LDSS3 is there
3961360
@feigewang
Magellan LDSS3 is there
688a8d9
@feigewang
Merge branches 'develop' and 'ldss3' of https://github.com/pypeit/PypeIt
70d085e 
 into ldss3

# Conflicts:
#	pypeit/scripts/view_fits.py
#	pypeit/spectrographs/util.py
@feigewang
clean up
bdd5d43
@feigewang
imacs is there
5c0de23
@feigewang
minor
cd1cee6
@feigewang
minor
56714de
@feigewang
Merge branch 'develop' of https://github.com/pypeit/PypeIt into ldss3
ea5786e 
� Conflicts:
�	pypeit/scripts/view_fits.py
@feigewang
minor
a3e4be2
@profxj
Merge remote-tracking branch 'origin/develop' into ldss3_dev
fdfe2ab
@profxj
fussin
d5427b0
@profxj
reading, running
9b74a2d
@profxj
minor
13f1438
@profxj
Merge remote-tracking branch 'origin/develop' into ldss3_dev
8036642
@profxj
arc fussin
ac96078
@profxj
mo
de7b61d
@feigewang
get ldss3 vph_red working.
ad909c9
@feigewang
Merge branch 'develop' of https://github.com/pypeit/PypeIt into ldss3…
cc8e459 
…_new

� Conflicts:
�	CHANGES.rst
�	pypeit/spectrographs/__init__.py
@gphil1
updating mmt_binospec.py
fa5eddd
@feigewang
Merge branch 'release' of https://github.com/feigewang/PypeIt into ld…
a53f5d3 
…ss3_new

# Conflicts:
#	deprecated/defs.py
@profxj
Merge pull request #2025 from feigewang/ldss3_new
4845fcc 
update ldss3_new branch
@gphil1 @feigewang
updating mmt_binospec.py
2ce4f46
@feigewang
Merge remote-tracking branch 'origin/mmt_binospec_v202508' into mmt_b…
fdbef14 
…inospec_v202508
@feigewang
make OSMOS working for the first TDE.
9d8ccbb
@feigewang
Merge branch 'ldss3_new' of https://github.com/feigewang/PypeIt into mdm
49f9322
@debora-pe
Merge branch 'bino_maskdef' into mdm
7b425b0
@debora-pe
Merge pull request #2051 from feigewang/mdm
8f05728 
merge our development for Binospec MOS and some other minor changes
@debora-pe
fix a conflict
8be6703
@debora-pe
remove some stuff
8a713f9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bpholden bpholden Awaiting requested review from bpholden

@kbwestfall kbwestfall Awaiting requested review from kbwestfall

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@debora-pe @feigewang @profxj @gphil1