Skip to content

Slitmaskgui communication #16

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
Closed
NominalAnxiety wants to merge 130 commits into from
Closed

Slitmaskgui communication #16

NominalAnxiety wants to merge 130 commits into from

Conversation

@NominalAnxiety
Copy link
Contributor

@NominalAnxiety NominalAnxiety commented Aug 26, 2025

You can now switch to see what the demo CSU looks in real-time! Click status to see the initial configuration.

  • auto updates bars in gui when configuring or calibrating the demo CSU
  • does not currently support running any edited mask configs

@NominalAnxiety
Early stage GUI for the CSU currently can import a starlist file and …
055ba83 
…display on a table the target list
@NominalAnxiety
initial commit
f0f3cd7
@NominalAnxiety
added a interactive image of the bars that you can select which will …
2457760 
…then make them change color
@NominalAnxiety
Added slits that display the star they are opening for and should cha…
503e2e6 
…nge their position when called
@NominalAnxiety
initial commit
ab8a9b8
@NominalAnxiety
made it so that the slit position table should now be able to change …
7058de5 
…the position of the slits on the image when the slit positions are entered
@NominalAnxiety
slightly edited the interactiveslitmask file to take out unneccesary …
9d129c9 
…code
@NominalAnxiety
Changed file names and made the files overall easier to read
66e1bc9
@NominalAnxiety
made a few changes that were inconsequential
024c698
@NominalAnxiety
Merge branch 'interactive_slit_mask'
50358cf
@NominalAnxiety
renamed the slit_mask to be nicer
02c703b
@NominalAnxiety
made it so that the interactive slit mask could change the slit posit…
e052d71 
…ions and the star names
@NominalAnxiety
Merge branch 'main' into slit_position_table
06e2230
@NominalAnxiety
messed up the merge so this is the last touches on the merge
eeeede3
@NominalAnxiety
Merge branch 'CaltechOpticalObservatories:main' into main
b3ff0bd
@NominalAnxiety
changed the name of the slit postion table file to make it look better
5f19058
@NominalAnxiety
the slit position table now interacts with the interactive slit mask …
4d81409 
…and visa versa
@NominalAnxiety
Merge remote-tracking branch 'origin'
a785ef7
@NominalAnxiety
updated the GUI layout to use QsizePolicy and updated other functions…
d2a7ae3 
… for Quality of life changes as well as add an if statement in the interactive slit mask
@NominalAnxiety
initial commit
b458ef5
@NominalAnxiety
forgot to commit but now I can get the offset from center data (the p…
8981748 
…rogram as a whole probably doesn't work)
@NominalAnxiety
added input to the row widget and the slit mask but none of it is act…
7529251 
…ually right but its kind of a proof of concept idk
@NominalAnxiety
when you press run, it will give a list of stars and show what slit p…
051720d 
…osition those stars would have (does not double slit)
@NominalAnxiety
fixed bug when selecting items in the interactive slit mask. Changed …
64474d1 
…the gaia query system. fixed the y value of stars so the correct slits are displayed
@NominalAnxiety
changed gitignore slightly so it is less annoying to commit
57d2473
@NominalAnxiety
added a green fov box on the slit mask and fixed the x coordinates
5827e15
@NominalAnxiety
Added titles to all the widgets
a9cff53
@NominalAnxiety
Everything can be resized by the user on a whim now (most of everything)
38c58a0
@NominalAnxiety
the slit_mask now scales when the area is changed by the user
54547f3
@NominalAnxiety
changed the display to have splitters so that pretty much everything …
15defd3 
…can be resized freely
@NominalAnxiety
Copy link
Contributor Author

NominalAnxiety commented Aug 27, 2025

updated the mask config widget. Now when you update the mask configuration you can configure the bars for that configuration.

@NominalAnxiety
added docstrings to waveband_view.py to improve clarity
33042e0
@NominalAnxiety
Merge branch 'slitmaskgui_wavelengthview' into slitmaskgui_communication
a63e69d
@NominalAnxiety
made code more readable
24148cd
@NominalAnxiety
added requirements.txt for slitmaskgui and a github workflow
17a3dd9
@NominalAnxiety
removed seme uneeded code and fixed a bug
936a31a
@NominalAnxiety
updated the tests and added a new one
6598f29
@NominalAnxiety
updated logic for slitmask_view
a82794b
@NominalAnxiety
fixed a bug that would lag out the app when you selected rows in slit…
663c179 
… position table (forgot to add parenthesis)
@NominalAnxiety
added to mask_config pytest. made one of the tests a lot faster and a…
be67bb4 
…dded another test
@NominalAnxiety
added some more tests
3a703ea
@NominalAnxiety
updated the tests and added a new file
63d26b7
@NominalAnxiety
removed a comment
7996861
@NominalAnxiety
thread that can check if it is connected to the internet or not
d4edd3c
@NominalAnxiety
if offline it will display offline on top of window
864870a
@NominalAnxiety
fixed a bug and added offline mode
93f1203
@NominalAnxiety
Merge remote-tracking branch 'origin' into slitmaskgui_offline_mode
37dae13
@NominalAnxiety
changed CI
f389cfe
@NominalAnxiety
changed CI
b6f80c5
@NominalAnxiety
changed CI again
9b970fd
@NominalAnxiety
changed CI and offline mode
a5b48dd
@NominalAnxiety
updated the requirements.txt
dc88531
@NominalAnxiety
CI works now (should)
fa4c11b
@NominalAnxiety NominalAnxiety deleted the slitmaskgui_communication branch September 15, 2025 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@prkrtg prkrtg prkrtg approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@NominalAnxiety @prkrtg