Skip to content

Consolidate likelihoods v2 - #306

Merged
fabern merged 7 commits into
masterfrom
consolidate-likelihoods_v2
Jul 29, 2025
Merged

Consolidate likelihoods v2#306
fabern merged 7 commits into
masterfrom
consolidate-likelihoods_v2

Conversation

@fabern

@fabern fabern commented Jul 25, 2025

Copy link
Copy Markdown
Member

Please merge #302 first.

This slightly modifies the calibration:

  • calibrate_sofun() now hands parameters as named vector to the cost functions.
  • updates the vignette on custom cost-function making use of this
  • previous cost functions are (for the moment) kept as-is (Future TODO: make use of named parameters)

Changes are fully backward compatible with the previous cost functions provided in the package.
It is possible (but highly unlikely) that previous custom cost functions show changed behavior, since in R named vectors behave almost exactly like unnamed ones.

Please merge #302 first. #302 will create v.5.2.0. #306 here will create v5.2.1

@fabern
fabern force-pushed the consolidate-likelihoods_v2 branch from af46d5b to 6977e09 Compare July 28, 2025 16:46

@fabern fabern left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed myself: good to merge.

@fabern
fabern marked this pull request as ready for review July 29, 2025 10:33
@fabern
fabern merged commit 7b145b3 into master Jul 29, 2025
7 checks passed
@fabern
fabern deleted the consolidate-likelihoods_v2 branch July 29, 2025 12:09
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