Codecov Report

❌ Patch coverage is 96.40719% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.19%. Comparing base (3adc64c) to head (2afce43).
⚠️ Report is 1 commits behind head on master.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
+ Coverage   89.36%   93.19%   +3.82%     
==========================================
  Files          30       32       +2     
  Lines        1693     1764      +71     
==========================================
+ Hits         1513     1644     +131     
+ Misses        180      120      -60     

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

Second question -- should all of the downloading functions return the same class as the output? Currently, imgw_telemetry returns a data table, and the rest (?) return data frames..

(The rest seems good)

Second question -- should all of the downloading functions return the same class as the output? Currently, imgw_telemetry returns a data table, and the rest (?) return data frames..

We did that in previous version, but for hydrological dataset simple transition for 1 full year of dataset from data.table to data.frame with as.data.frame() for approx 50,000,000 rows takes ages and requires lot of RAM. Also data.table usually has both classes: data.frame and data.table, so theoretically we still under the same umbrella.

one question -- should there be a possibility to call the meteo_imgw_datastore function from meteo_imgw (similarly to meteo_imgw_daily, etc.)?

It will be addressed in a seperate issue #112