Skip to content

add nexrad source functionality#177

Merged
bart1 merged 36 commits into
mainfrom
feature/source_birdcast_archive
Jun 4, 2026
Merged

add nexrad source functionality#177
bart1 merged 36 commits into
mainfrom
feature/source_birdcast_archive

Conversation

@iskandari
Copy link
Copy Markdown
Contributor

@iskandari iskandari commented May 10, 2026

Adds support for retrieving NEXRAD-based VPTS data in getRad with:

  • nexrad as a new source in get_vpts() and get_vpts_coverage()
  • get_vpts_nexrad() and get_vpts_coverage_nexrad() helpers
  • integration with the public BirdCast S3 VPTS archive
  • radar/date coverage validation
  • caching support for NEXRAD coverage metadata
  • comprehensive tests for coverage retrieval, validation, and VPTS downloads
  • updated documentation and examples for NEXRAD usage
  • incidentally fixes the Estonia PVOL test by making the download failure-tolerant for intermittent 429/500 API responses.

@iskandari iskandari changed the title Feature/source birdcast archive add nexrad archive functionality May 10, 2026
@iskandari iskandari changed the title add nexrad archive functionality add nexrad source functionality May 10, 2026
@iskandari iskandari requested review from adokter and removed request for adokter May 10, 2026 04:24
@codecov
Copy link
Copy Markdown

codecov Bot commented May 10, 2026

Codecov Report

❌ Patch coverage is 96.77419% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.43%. Comparing base (cf6e1d7) to head (c899780).

Files with missing lines Patch % Lines
R/get_vpts_aloft.R 66.66% 2 Missing ⚠️
R/zzz.R 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #177      +/-   ##
==========================================
- Coverage   92.47%   92.43%   -0.04%     
==========================================
  Files          28       28              
  Lines        2112     2048      -64     
==========================================
- Hits         1953     1893      -60     
+ Misses        159      155       -4     

☔ View full report in Codecov by Harness.
📢 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.

@iskandari iskandari requested a review from peterdesmet May 10, 2026 14:45
@bart1
Copy link
Copy Markdown
Collaborator

bart1 commented May 11, 2026

Hi @iskandari Great to see all activity here! I hope to have some time later to actually also test this! For my reference this is a different dataset from #175 right? Or is it the same data?

@iskandari
Copy link
Copy Markdown
Contributor Author

iskandari commented May 11, 2026

@bart1 it is a different dataset that we recently made public on S3

@adokter
Copy link
Copy Markdown

adokter commented May 11, 2026

This is the vol2bird/bioRad dataset

@adokter
Copy link
Copy Markdown

adokter commented May 11, 2026

@bart1 @peterdesmet please coordinate with me on the exact timing of merging/releasing this - good to have this available before the IRAC workshop but not fully unannounced.

@adokter adokter self-requested a review May 11, 2026 18:37
@bart1
Copy link
Copy Markdown
Collaborator

bart1 commented May 12, 2026

@bart1 it is a different dataset that we recently made public on S3

Thanks for the update, good to understand the differences!

@bart1
Copy link
Copy Markdown
Collaborator

bart1 commented May 18, 2026

It looks good for integrate. I will do some testing soon. One concern I have is the source argument to get_vpts. Currently you use nexrad however if we also integrate the "dark ecology" dataset (#175) this might become confusing. @iskandari Is it maybe better to use birdcast as a source name?

I'm anyway still puzzling how to integrated the "dark ecology" dataset as it will be an other path to disk, like we now have the local source. The file structure is however different so we might need an extra argument for the local file structure (were still somewhat flexible here as we have not released this yet). @PietrH any thoughts on this?

Comment thread R/get_vpts_nexrad.R Outdated
@bart1 bart1 self-requested a review May 19, 2026 12:08
@bart1
Copy link
Copy Markdown
Collaborator

bart1 commented May 21, 2026

@adokter / @iskandari any objection to renaming to birdcast to avoid confusion with other nexrad derived datasets?

Copy link
Copy Markdown
Collaborator

@bart1 bart1 left a comment

Choose a reason for hiding this comment

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

I tested it

@bart1 bart1 merged commit 0dd2fc9 into main Jun 4, 2026
11 of 12 checks passed
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.

3 participants