Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Merge dev for 1.0 release #112

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
Merged
merged 272 commits into from
Nov 9, 2022
Merged

Merge dev for 1.0 release #112

merged 272 commits into from
Nov 9, 2022

Conversation

@DM1122
Copy link
Member

@DM1122 DM1122 commented Nov 1, 2022

No description provided.

gjerazik and others added 30 commits February 20, 2022 12:37
@gjerazik
add new plotting format
74959fd
@stephanieyflu
Updated plots
b20843e
@stephanieyflu
Corrected foreoptic to slit spacing
0970b54
@stephanieyflu
Revised plots
de9f01e
@stephanieyflu
Revised plots
c7a1bf1
@stephanieyflu
Reordered plots
0fbc4f5
@DM1122
Fix some snr bugs
27c7db4
@jenniferzzhang
Update pipe_gsd_swath.py
a98f441 
changed variable names, readability, region plots
@DM1122
Package update
b219440
@DM1122
Fix pre-commit black hook python version
a359d59
@rararani
completed ambient temperature pipeline
bfb0917
@rararani
commiting edited files for ambient temp pipeline
5f85f8a
@gjerazik
Add plot code
ea142d3
@DM1122
Format
abf6adc
@gjerazik
Change micron units
02bdcdf
@DM1122
Fix grism efficiency pipeline framework
afc2624
@DM1122
Add epsilon term to snr function
b1dac4d
@rararani @DM1122
Update payload_designer/components/basecomponent.py
938ee04 
Co-authored-by: David Maranto <[email protected]>
@rararani @DM1122
Update payload_designer/components/basecomponent.py
0370732 
Co-authored-by: David Maranto <[email protected]>
@gjerazik
very minor change
39d3590
@gjerazik
Add final pipelines
08a90aa
@gjerazik
Add final pipeline 2
d491cff
@DM1122
Update project to newest poetry-template standard
b31f55a
@DM1122
Merge pull request #76 from spacesys-finch/efficiency_pipeline
f032f2e 
Efficiency pipeline
@DM1122
Merge pull request #75 from spacesys-finch/feature/temperature
d2044ac 
Feature/temperature: Ambient temperature pipeline
@DM1122
Merge pull request #72 from spacesys-finch/pipeline-gsd-swath
af192c2 
Pipeline GSD, Swath
@DM1122
Merge branch 'dev' into mass-and-volume-envelopes-pipeline
fa186b6
@DM1122
Merge pull request #71 from spacesys-finch/mass-and-volume-envelopes-…
b92cf10 
…pipeline

Mass and volume envelopes pipeline
@DM1122
Merge branch 'dev' into grism-angular-range
4565c4d
@DM1122
Merge pull request #70 from spacesys-finch/grism-angular-range
738837f 
Add pipeline for grism angle
DM1122 and others added 19 commits November 1, 2022 21:15
@DM1122
Add pipeline readme
8ea7a18
@KevinYuann
Added offset to ground target error tradebook
2cd95dc
@DM1122 @jenniferzzhang
Split signal into subfunctions
f775d24 
Co-authored-by: jenniferzzhang <[email protected]>
@KevinYuann
Updated ground target error tradebook and tests
349552e
@DM1122
Add config for gource
f293994
@DM1122
Add poetry toml file to house virtualenvs.in-project config
8d0a7ec
@DM1122
Add cyclic-import to ignore list
fa9dd32
@KevinYuann
Fixed the function and made code easier to read
0d1666c
@KevinYuann
Output is now in [m] instead of [km]
bdfe502
@KevinYuann
moved the unit conversion to the function
dd5de33
@KevinYuann
Merge pull request #114 from spacesys-finch/Ground-Target-Error-Plus-…
130b964 
…Offset-Tradebook

Edited Ground Target Error Tradebook to Include Offset
@jenniferzzhang
Added waveband as Sensor param and to signal_light
10aed89
@DM1122
Re-add waveband parameter to SNR tests
4d16854
@jenniferzzhang
SNR unit tests passed
9c3dc79
@jenniferzzhang
SNR tradebook fixed
00ef8be
@DM1122
Update ground target error trabebook
4a4bb2b
@DM1122
Remove duplicate tradebook
d315043
@DM1122
Cleanup tradebook
5883e46
@DM1122
Merge pull request #117 from spacesys-finch/snr-unit-tests-refactored
0093f2c 
Snr unit tests refactored
@DM1122 DM1122 requested a review from sxu0 November 8, 2022 19:04
@DM1122 DM1122 added this to the V1.0 milestone Nov 8, 2022
@DM1122 DM1122 marked this pull request as ready for review November 8, 2022 22:40
sxu0
sxu0 previously approved these changes Nov 9, 2022
View reviewed changes
Copy link
Contributor

@sxu0 sxu0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what a milestone, super proud of y'all!

@DM1122 DM1122 merged commit 7f7244a into main Nov 9, 2022
@DM1122 DM1122 deleted the dev branch November 9, 2022 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sxu0 sxu0 sxu0 left review comments

Assignees

@DM1122 DM1122

Labels

None yet

Projects

None yet

Milestone

V1.0

Development

Successfully merging this pull request may close these issues.

10 participants

@DM1122 @sxu0 @gjerazik @stephanieyflu @jenniferzzhang @rararani @max01110 @abeerfatima @KevinYuann