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

Merge dev and do new release #386

Merged
merged 124 commits into from
Aug 26, 2024
Merged

Merge dev and do new release #386

merged 124 commits into from
Aug 26, 2024

Conversation

vhaasteren
Copy link
Member

@vhaasteren vhaasteren commented May 9, 2024 •
edited
Loading

As described in issue #385, merge dev->master and do a new release

All other changes postponed for merge into master. Those will get updated on the new dev branch

vhaasteren and others added 30 commits April 21, 2023 17:55
@vhaasteren
First addition of FastShermanMorrison for testing
20f0e20
@vhaasteren
Added numpy and cython extensions to setup.py
312093b
@vhaasteren
Updated docstrings and method selector for fast-sherman-morrison
1b536bc
@vhaasteren
Merge remote-tracking branch 'github-vhaasteren/fastshermanmorrison' …
dff897e 
…into fastshermanmorrison
@vhaasteren
Added numpy and cython to requirements_dev.txt
db81f64
@vhaasteren
Moved FastShermanMorrison to external package
d8e1a35
@vhaasteren
Removed fastshermanmorrison from setup.py
fb7f533
@vhaasteren
Conform to black linter
50cc052
@vhaasteren
Added test for fast-sherman-morrison
39b6b0e
@vhaasteren
Added requirements_tests.txt, so that the extensions can be loaded fo…
0e3435c 
…r testing.
@vhaasteren
Added decorator to show fastshermanmorrison-pulsar warning only once
60a117e
@vhaasteren
Proper linting corrections
851b13e
@vhaasteren
Changed the EcorrKernelWarning machanism
bf1705e
@vhaasteren
Blacking
f8d71f5
@vhaasteren
Lint update
8c4bfa8
@vhaasteren
Removed the have_shown stuff for now
fb01136
@vhaasteren
Placed fastshermanmorrison into requirements_dev.txt
3ff9921
@vhaasteren
Added Percentage Point Function for all parameters
ef663fa
@vhaasteren
Added PPF tests
1fa8dd9
@vhaasteren
Done linting on parameter.py and test_parameter.py
61599a8
@vhaasteren
Allow for index arrays in ShermanMorrison, so we do not need sorted T…
f78af6b 
…OAs for EcorrKernelNoise signals
@vhaasteren
Updated all the white signals and block matrix code for index array i…
4d678ae 
…ndexing
@vhaasteren
Provided mechanism for showing warning message about fastshermanmorri…
37b4c9c 
…son, and updated fastshermanmorrison-pulsar in requirements_dev.txt
@vhaasteren
Added pragmas for no coverage of exception results
1df974a
@vhaasteren
Linting: forgot a space in a pragma
3698716
@vhaasteren
Modified pulsar.py to allow for additional pulsar types from external…
ca9a5a9 
… packages
@vhaasteren
Moved drop t2pulsar code
6703327
@vhaasteren
Bugfix for bug #358
d959b7a
@vhaasteren
Linting
860e2b6
@vhaasteren
Added tests for scrambled/unsorted IPTA datasets for EcorrKernelNoise
96af0ba
@vhaasteren vhaasteren marked this pull request as draft May 9, 2024 14:48
@AaronDJohnson
Copy link
Collaborator

I think we need to merge main into dev to get the macos tests to work.

@AaronDJohnson
Copy link
Collaborator

AaronDJohnson commented May 9, 2024 •
edited
Loading

Ah maybe not. It's the usual suspect:
Cannot find tempo2 install location.

@vhaasteren
Copy link
Member Author

@AaronDJohnson,

It turns out that enterprise_extensions has some dependencies on the dev branch. Is it an idea to just merge dev as-is to accommodate, and deal with tempo2/pint optional dependencies and feather later? Those can go into the dev branch when ready.

@AaronDJohnson
Copy link
Collaborator

@vhaasteren I think that's a good idea. Lately, I have been more busy than I thought I would be. So let's go ahead and merge and then work on these extra features.

@AaronDJohnson
Copy link
Collaborator

AaronDJohnson commented Aug 4, 2024 •
edited
Loading

The tests weren't passing because of libstempo/TEMPO2.... Those issues should be solved now. Maybe the tests will pass on rerunning them?

@AaronDJohnson AaronDJohnson marked this pull request as ready for review August 12, 2024 14:56
@AaronDJohnson
Copy link
Collaborator

Two more issues now: TEMPO2 can't be found on the Mac installs and 3.7 is still required for merging.

AaronDJohnson and others added 17 commits August 13, 2024 07:50
@AaronDJohnson
move to macos-13
9b41516
@AaronDJohnson
Merge branch 'dev' into fix_ci
20acc8d
@AaronDJohnson
add 3.12, update codecov
9863f8c
@AaronDJohnson
Merge branch 'fix_ci' of https://github.com/AaronDJohnson/enterprise …
958670c 
…into fix_ci
@AaronDJohnson
Add Python 3.12
bad8a1a
@AaronDJohnson
undo codecov upgrade
894b441
@AaronDJohnson
redo upgrade codecov, lower max healpy version
0a5fea8
@AaronDJohnson
spacing is important...
ac45bba
@AaronDJohnson
missed one...
e75c28d
@AaronDJohnson
undo upper limit on healpy
ef756ad
@alyssacassity
Update selections.py for "CHIME"
2096868 
Add "CHIME" to 
def nanograv_backends(backend_flags):
@alyssacassity
Update selections.py
25d4747 
added "VEGAS" to list as requested
@AaronDJohnson
try to get verbose output
0cee27c
@AaronDJohnson
attempt 7896868
70df5ec
@AaronDJohnson
try to skip the problematic test
4f6154a
@vhaasteren
Merge pull request #391 from AaronDJohnson/fix_ci
a24ed09 
Fix CI
@AaronDJohnson
Merge pull request #393 from alyssacassity/patch-1
508e0d9 
Update selections.py for "CHIME" and "VEGAS"
@AaronDJohnson AaronDJohnson merged commit 1ab8ad9 into master Aug 26, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AaronDJohnson AaronDJohnson AaronDJohnson approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@vhaasteren @AaronDJohnson @vallis @paulthebaker @siyuan-chen @alyssacassity