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.

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

Noxcal prepare release #481

Merged
merged 28 commits into from
Feb 12, 2024
Merged

Noxcal prepare release #481

merged 28 commits into from
Feb 12, 2024

Conversation

o-smirnov
Copy link
Collaborator

No description provided.

JSKenyon and others added 27 commits September 20, 2019 09:12
* partial fix for #93. Should allow for one spw at a time

* fixes #93. Adds workaround for casacore/python-casacore#130

* added future-fstrings

* correctly named future-fstrings

* ...and install it for all pythons

* back to mainstream sharedarray

* lowered message verbosity

* droppping support for python<3.6

* Issues 427 and 429 (#430)

* Fixes #427

* Fixes #429

* Update __init__.py

Bump version

* Update Jenkinsfile.sh (#447)

* Update Jenkinsfile.sh

Remove python 2 building

* Update Jenkinsfile.sh

* Issue 431 (#432)

* Beginning of nobeam functionality.

* Remove duplicated source provider.

* Change syntax for python 2.7 compatibility.

* Update montblanc version in setup to avoid installation woes.

Co-authored-by: JSKenyon <jonosken@gmail.com>

* Remove two uses of future_fstrings.

* make pypi compatible (#446)

* make pypi compatible

* error

* Update setup.py

Only support up to v0.6.1 of montblanc

* Update setup.py

Co-authored-by: Benjamin Hugo <bennahugo@gmail.com>
Co-authored-by: JSKenyon <jskenyon007@gmail.com>
Co-authored-by: JSKenyon <jonathan.simon.kenyon@gmail.com>

* Update __init__.py

* Fix warning formatting

Co-authored-by: Oleg Smirnov <osmirnov@gmail.com>
Co-authored-by: Benjamin Hugo <bennahugo@gmail.com>
Co-authored-by: JSKenyon <jskenyon007@gmail.com>
Co-authored-by: JSKenyon <jonosken@gmail.com>
Co-authored-by: Gijs Molenaar <gijs@pythonic.nl>
Co-authored-by: JSKenyon <jonathan.simon.kenyon@gmail.com>
* fixes to AIPS leakage plotter (more table format support); started an AIPS bandpass plotter but this is WIP

* add check for INDE value in PRTAB leakages

* changed to hierarchical config
@JSKenyon
Copy link
Collaborator

JSKenyon commented Feb 12, 2024

Wait, something is off here. The noxcal branch was merged - why is the diff on this PR so large? Maybe merging in master/rebasing on master would be sensible?

Edit: It may just be that this was branched a long time ago.

@o-smirnov
Copy link
Collaborator Author

Yeah I'm not sure what's the deal. Did you make this branch, @JSKenyon? Or did @Athanaseus?

@o-smirnov
Copy link
Collaborator Author

Actually looks like the diff is almost entirely due to the stimelation branch we just merged in...

@JSKenyon
Copy link
Collaborator

Actually looks like the diff is almost entirely due to the stimelation branch we just merged in...

I think by merging in master, the diff was minimized. Does this supercede #458?

Copy link
Collaborator

@JSKenyon JSKenyon left a comment

Choose a reason for hiding this comment

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

LGTM!

@o-smirnov o-smirnov merged commit 5686a1d into master Feb 12, 2024
1 check passed
@o-smirnov o-smirnov deleted the noxcal-prepare-release branch February 12, 2024 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants