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

Py311 #80

Open
wants to merge 95 commits into
base: main
Choose a base branch
from
Open

Py311 #80

wants to merge 95 commits into from

Conversation

scfleming
Copy link
Collaborator

Modernizes up to Python 3.12 as well as Python 3.11, despite the branch name.

jaymedina and others added 30 commits October 11, 2022 12:09
@jaymedina
initial commit of spectra preview sonification
81e589f
@jaymedina
.
e2bb7a5
@scfleming
major edit on sonify_preview function, defines amplitude based on are…
9ed0116 
…a-under-curves, tremolo values based on std dev
@jaymedina
.
8fa6db0
@jaymedina
connecting classes
78e8805
@jaymedina
updating play() with layered pitches that are delayed at different times
2c01180
@jaymedina
multiplying amplitude by factor to increase vol
4ef0c86
@jaymedina
some updates to duration and delay periods
b982c88
@jaymedina
Constraint added to keep tremolo values at or below 15
cb7bca3
@jaymedina
added some todo items to preview_play
95f2af5
@jaymedina
Adding a "tremolo factor" which will multiply all tremolo vals by 10
3810d61
@jaymedina
modulating amplitudes with inverse log; playing notes separately;
6bbcd07
@jaymedina
snapshots notebook
792f6ce
@jaymedina
replacing std with the std error of a linear regression fit
15ae515
@jaymedina
normalizing amplitudes with the max amp for more comfortable hearing …
f9bdd12 
…range; setting default lfo to min amp for ranges with 0 freq
@jaymedina
changing default lfo to 1.0
7d567fb
@jaymedina
minor changes
72d800b
@jaymedina
initial attempt at increasing amplitude for non-oscillating behavior.…
4f2e4cd 
… i.e. zero tremolo, non-zero amp
@jaymedina
adding verbose and plotting keyword args to sonify_preview
0a59077
@srisund
only left channel sounding bug fixed
911703c
@srisund
Real spectra demo notebook added
55cfda4
@scfleming
WIP: flipped colors in plots to match wavelength order, added two mod…
c41c9c0 
…es for preview type = scan and ensemble, ensemble plays with diff pitches and the combined, scan plays with same pitch and no combined at the end, re-wrote OBAFGKM notebook so it fetches sample spectra from public Box links at STScI, no assumptions made about location on user's computer, still need to finish the BAFGKM stars, but both versions for O star work
@jaymedina
updating .gitignore with new generated directory miles_stellar_spectra
836f458
@jaymedina
adding requests and notebook as dependencies of astronify. pinning re…
b69b2f0 
…quests version to 2.28.*
@scfleming
intermediate checkin
5bf6b63
@scfleming
after merge
b080e8f
@scfleming
removing thinkx, numpy 2.0 fix, updating versions of libraries used i…
95f5d6e 
…n tox test builds
@scfleming
fixing documentation format error
c3c19ac
@scfleming
adding new requireemnts in RTD yaml
f162357
@scfleming
fixing RTD yaml
005732a
@scfleming
more lynt iterating
4430c8d
@scfleming
lynting iteration
d308938
@scfleming
lynting iteration
a79ef4a
@scfleming
lynting series.py
0282c6f
@scfleming
lynting series.py
0e1397f
@scfleming
more lynting
87139b7
@scfleming
lynt iterating
c008d9a
@scfleming
another round of lynting
00c0f72
@scfleming
almost last round of lynting
e39226e
@scfleming
last lynting
1006608
@scfleming
adding Pey-Lian as a contributing author
79a6aa3
@scfleming
adding CODEOWNERS file so merges into main require approval from at l…
b4883e2 
…east one code owner
@scfleming
splitting developer documentation from main doc, adding some links to…
e589c78 
… tutorials in case people are coming from here and not the web homepage
@scfleming
trying to fix links
3800b43
@scfleming
trying to fix links
34b37fb
@scfleming
adding better tip of how to build docs locally with sphinx directly
b359932
@scfleming
adding some work-in-progress dev steps, adding __version__, updating …
40a5b70 
…change log
@scfleming
Preparing release 0.11
89603bb
@scfleming
Preparing release 0.11
70aa31a
@scfleming
Preparing release 0.11
f8bf669
@scfleming scfleming linked an issue Jan 31, 2025 that may be closed by this pull request
ANN: Please migrate away from pytest-openfiles #71
Open
@scfleming scfleming requested a review from ceb8 January 31, 2025 20:41
@scfleming
Copy link
Collaborator Author

@ceb8 I had to fix some "at the very end" merge conflicts. I think I have done so correctly, and things seem to be passing. Although if possible it would be great to confirm further that things are working, especially regarding the case-sensitivity of columns when specifying time and value names. The intention is to be for them to work whether the column names are all lowercase or not.

@scfleming scfleming linked an issue Jan 31, 2025 that may be closed by this pull request
Installing astronify at the Rubin Science Platform in data.lsst.cloud #74
Open
@scfleming
Copy link
Collaborator Author

I did a quick test of use of column names with not-all-lowercase, on what should have been a fresh install, and it seemed to work.

@scfleming scfleming linked an issue Jan 31, 2025 that may be closed by this pull request
Update installation instructions. #77
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ceb8 ceb8 Awaiting requested review from ceb8

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@scfleming @jaymedina @srisund