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

Add S350 flux value #113

Merged
merged 1 commit into from
Sep 8, 2022
Merged

Conversation

mattpitkin
Copy link
Owner

As of v1.68 (or maybe from 1.66/67 onwards), so pulsar have an S350 flux value. This can now be extracted from a psrqpy query. Note that there are no pulsars for which the S30, S10G, S20G and S50G are added yet, so these are not listed as parameters available within psrqpy yet.

@mattpitkin mattpitkin linked an issue Sep 7, 2022 that may be closed by this pull request
@pep8speaks
Copy link

Hello @mattpitkin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 153:21: E222 multiple spaces after operator

@codecov
Copy link

codecov bot commented Sep 8, 2022

Codecov Report

Merging #113 (0e21060) into master (e50e3a2) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
- Coverage   75.22%   75.17%   -0.06%     
==========================================
  Files           7        7              
  Lines        3193     3194       +1     
  Branches      560      560              
==========================================
- Hits         2402     2401       -1     
- Misses        570      571       +1     
- Partials      221      222       +1     
Impacted Files Coverage Δ
psrqpy/pulsar.py 47.61% <ø> (ø)
psrqpy/config.py 100.00% <100.00%> (ø)
psrqpy/search.py 75.86% <0.00%> (-0.14%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@mattpitkin mattpitkin merged commit e0b7b76 into master Sep 8, 2022
@mattpitkin mattpitkin deleted the 111-add-some-more-radio-flux-frequencies branch September 8, 2022 13: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.

Add some more radio flux frequencies
2 participants