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

Baseline patch #278

Merged
merged 40 commits into from
Jun 28, 2024
Merged

Baseline patch #278

merged 40 commits into from
Jun 28, 2024

Conversation

teuben
Copy link
Collaborator

@teuben teuben commented Jun 27, 2024

No description provided.

teuben and others added 30 commits February 20, 2024 10:05
@teuben
add a __main__ entry
41617ce
@pre-commit-ci
[pre-commit.ci] pre-commit autoupdate
d030364 
updates:
- [github.com/psf/black: 24.4.0 → 24.4.2](psf/black@24.4.0...24.4.2)
@mpound
Merge branch 'main' into pre-commit-ci-update-config
59206cd
@teuben
Merge branch 'main' of https://github.com/GreenbankObservatory/dysh
aea5c02
@teuben
normalize freq axis from 0..nchan-1 to avoid roundoff
22250dc
@teuben
note on a possible future enhancement for a min. match routine
6b90658
@teuben
proper early return
7a79163
@teuben
use the new minimum_string_match() function
a3aa507
@teuben
implement minimum_string_match()
b5d7191
@teuben
remove debug
f53dd8f
@teuben
add warning
6435891
@teuben
add test for minimum_string_match
3e06d9c
@teuben
add TGBT21A_501_11.raw.vegas.fits
779a871
@teuben
allow include= in baseline selection (but then exclude= is ignored)
f055b7a 
only works in channel space for now
@teuben
my hotchpotch of spyder cells to test getps() and getfs()
e91163a
@teuben
Merge remote-tracking branch 'origin/release-0.3.0' into baseline-patch
87f6184
@teuben
add examples, clarify some caveats
43ce3dc
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
feaee14 
for more information, see https://pre-commit.ci
@mpound
Merge branch 'main' into pre-commit-ci-update-config
e5c83a2
@mpound
Merge pull request #234 from GreenBankObservatory/pre-commit-ci-updat…
5faa5e1 
…e-config

[pre-commit.ci] pre-commit autoupdate
@mpound
create integration number when loading SDFITS data
37a559c
@mpound
restore index file for testing
9fe4d2b
@mpound
remove big index file
7c76e99
@mpound
use smaller A6 file
2e56ca5
@mpound
Merge branch '272-integration-number-needs-to-be-defined-in-sdfits-in…
2e7bcd0 
…dex' of github.com:GreenBankObservatory/dysh into 272-integration-number-needs-to-be-defined-in-sdfits-index
@mpound
Merge pull request #273 from GreenBankObservatory/272-integration-num…
2bc1842 
…ber-needs-to-be-defined-in-sdfits-index

create integration number when loading SDFITS data
@teuben
Merge branch 'main' into baseline-patch
da85da1
@teuben
dummy patch
43b49e7
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
3cbc09d 
for more information, see https://pre-commit.ci
@teuben
another dummy patch to make it into 0.3.0
4cd4f74
@teuben teuben requested a review from mpound June 27, 2024 00:26
mpound
mpound reviewed Jun 27, 2024
View reviewed changes
src/dysh/spectra/core.py Outdated Show resolved Hide resolved
@teuben
Copy link
Collaborator Author

teuben commented Jun 28, 2024

I still need to commit the example notebook, it was expanded a lot. but also ran into a bug where I could not take the average of two spectra. to be resolved.

@teuben teuben merged commit 8506db4 into release-0.3.0 Jun 28, 2024
14 checks passed
@astrofle astrofle deleted the baseline-patch branch September 3, 2024 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@astrofle astrofle astrofle left review comments

@mpound mpound mpound 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
@teuben @astrofle @mpound