Skip to content

Merge GAIA BP RP data - #174

Open
Chamani8 wants to merge 2 commits into
karllark:masterfrom
Chamani8:master
Open

Merge GAIA BP RP data#174
Chamani8 wants to merge 2 commits into
karllark:masterfrom
Chamani8:master

Conversation

@Chamani8

Copy link
Copy Markdown
Contributor

New script to merge Gaia BP RP data into format that is compatible to be used with measure_extinction package. This script assumes that the original Gaia BP RP data for all the stars are stored in a single fits file, then follows the generic method to merge the BP and RP fluxes for each star.

@codecov

codecov Bot commented Apr 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.92%. Comparing base (fa2ac69) to head (97c1acd).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #174   +/-   ##
=======================================
  Coverage   47.92%   47.92%           
=======================================
  Files           7        7           
  Lines        1932     1932           
=======================================
  Hits          926      926           
  Misses       1006     1006           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@karllark

Copy link
Copy Markdown
Owner

Please fix the codestyle error.

@karllark

Copy link
Copy Markdown
Owner

I'm wondering what the plot looks like with this code. If I read things right, the code is going to create many fits files, two for each star. The versions of this code for other data are for a single star, so the plot then is straightforward to understand. Is the plot a lot of spectra? If so, does it make sense to include the plotting code?

@Chamani8

Copy link
Copy Markdown
Contributor Author

The plotting code is meant to make one plot per star, and save it rather than show it when the script is run. You are correct then it makes two plots per star, one for BP and another for RP, but it does not make overplot the stars in a single figure, rather it makes a separate plot for each star. I think the plotting is still useful to go back and check what the script did.

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.

2 participants