search

nlmixr2 / rxode2et

https://nlmixr2.github.io/rxode2et/
1 stars 1 forks source link

Added some prints to see which column is causing the issue #40

Closed mattfidler closed 5 months ago

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 92.85714% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 62.99%. Comparing base (25fae15) to head (8b4a88e).

Files Patch % Lines
src/et.cpp 92.85% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #40 +/- ## ========================================== + Coverage 62.16% 62.99% +0.83% ========================================== Files 21 21 Lines 4028 4035 +7 ========================================== + Hits 2504 2542 +38 + Misses 1524 1493 -31 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

billdenney commented 5 months ago

What was the underlying issue? It seems like it was something about NA time, but I know that there were already checks for that, so it must have been something more specific, I guess.

mattfidler commented 5 months ago

It was na time in the importing of event tables which was not handled.