Skip to content

Fix issue 188 2#251

Merged
kkappler merged 7 commits intotfkernelfrom
fix_issue_188_2
Apr 8, 2023
Merged

Fix issue 188 2#251
kkappler merged 7 commits intotfkernelfrom
fix_issue_188_2

Conversation

@kkappler
Copy link
Collaborator

@kkappler kkappler commented Apr 7, 2023

No description provided.

kkappler added 4 commits April 7, 2023 14:16
In this commit is a hodgepodge of experiments aimed at setting the
start/end times in mt_metadata for the synthetic data.

Many lines are from the old fix_issue_188 branch which has not been
looked at in some time, and not all are needed.

These will be cleaned up in following commits

[Issue(s): #188]
@codecov
Copy link

codecov bot commented Apr 7, 2023

Codecov Report

Merging #251 (0e0ebd4) into tfkernel (97e2c2d) will increase coverage by 60.85%.
The diff coverage is 91.54%.

@@              Coverage Diff              @@
##           tfkernel     #251       +/-   ##
=============================================
+ Coverage     18.07%   78.92%   +60.85%     
=============================================
  Files            72      103       +31     
  Lines          4017     5798     +1781     
=============================================
+ Hits            726     4576     +3850     
+ Misses         3291     1222     -2069     
Impacted Files Coverage Δ
tests/synthetic/test_multi_run.py 97.36% <ø> (ø)
aurora/general_helper_functions.py 61.29% <71.42%> (+11.29%) ⬆️
tests/test_general_helper_functions.py 93.75% <92.85%> (+93.75%) ⬆️
...ts/synthetic/test_metadata_values_set_correctly.py 94.11% <94.11%> (ø)
aurora/test_utils/synthetic/make_mth5_from_asc.py 83.47% <100.00%> (ø)
aurora/test_utils/synthetic/station_config.py 100.00% <100.00%> (ø)

... and 78 files with indirect coverage changes

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

@kkappler kkappler merged commit afc0e5f into tfkernel Apr 8, 2023
@kkappler kkappler deleted the fix_issue_188_2 branch April 8, 2023 00:57
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.

1 participant