Refactor likelihood module

hmgaudecker commented 2 months ago

In maximization_inputs.py, import from two modules with likelihood functions.
Remove the _jax-suffixes from likelihood functions, seems like a leftover from transition days.

review-notebook-app[bot] commented 2 months ago

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 91.39785% with 8 lines in your changes missing coverage. Please review.

Please upload report for BASE (reduce-mem-usage@4e64952). Learn more about missing BASE report.

Files with missing lines	Patch %	Lines
src/skillmodels/maximization_inputs.py	90.12%	8 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## reduce-mem-usage #77 +/- ## =================================================== Coverage ? 91.43% =================================================== Files ? 40 Lines ? 3070 Branches ? 0 =================================================== Hits ? 2807 Misses ? 263 Partials ? 0 ```

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

OpenSourceEconomics / skillmodels

Refactor likelihood module #77

Codecov Report