Fix for edge case when likelihood is infinite when profiling

[schuemie]

The current code only checks if the log likelihood is NaN, but under some circumstances it can be -Inf, causing a 'missing value where TRUE/FALSE needed' error.

I added a fix and some unit tests that test both scenarios (NaN or -Inf).

[codecov]

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (a2b3a95) 79.27% compared to head (14c8289) 79.24%.

❗ Current head 14c8289 differs from pull request most recent head 6527620. Consider uploading reports for the commit 6527620 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop      #72      +/-   ##
===========================================
- Coverage    79.27%   79.24%   -0.04%     
===========================================
  Files           56       56              
  Lines         7296     7298       +2     
===========================================
- Hits          5784     5783       -1     
- Misses        1512     1515       +3     

Files	Coverage Δ
R/ModelFit.R	87.08% <42.85%> (+0.71%)	⬆️

... and 2 files with indirect coverage changes

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

[msuchard]

code looks good and passes all tests!

[schuemie]

@msuchard could we have a release with this fix implemented? The current develop branch seems functional.