Skip to content

Conversation

@bart1
Copy link
Collaborator

@bart1 bart1 commented Nov 18, 2025

The implementation for local data reading is quite flexible to for example make it possible to also read daily data. It is mostly controlled through a glue formatting string as documented (please check if you understand).

Currently if no file is found the function fails. If some data is found it warns about the data not found. However it could be whole radars are absent and thus not read with only a warning

@bart1 bart1 linked an issue Nov 18, 2025 that may be closed by this pull request
@bart1 bart1 requested a review from PietrH November 18, 2025 16:13
@bart1 bart1 marked this pull request as ready for review November 18, 2025 17:02
@bart1
Copy link
Collaborator Author

bart1 commented Nov 18, 2025

@PietrH or @peterdesmet can one of you look if this works for you? I think this setup balances flexibility with usability but it would be good to see if it also works some what natural for you

@codecov
Copy link

codecov bot commented Dec 2, 2025

Codecov Report

❌ Patch coverage is 70.52632% with 28 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.41%. Comparing base (355775a) to head (681a6e5).

Files with missing lines Patch % Lines
R/zzz.R 0.00% 28 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #147      +/-   ##
==========================================
- Coverage   93.72%   92.41%   -1.32%     
==========================================
  Files          24       25       +1     
  Lines        1801     1859      +58     
==========================================
+ Hits         1688     1718      +30     
- Misses        113      141      +28     

☔ 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.

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.

Using local files as source in get_vpts?

2 participants