Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix behavior of weights in global fitting #171

Merged
merged 11 commits into from
May 4, 2021
Merged

Conversation

luisfabib
Copy link
Collaborator

Corrects the behavior of weights in global fitting when specifying them and some related issues with scaling of the datasets.

Closes #168

@luisfabib luisfabib added the bugfix Patches something that isn't working label May 3, 2021
@luisfabib luisfabib added this to the 0.14.0 milestone May 3, 2021
@luisfabib luisfabib requested a review from stestoll May 3, 2021 11:19
@luisfabib luisfabib self-assigned this May 3, 2021
@luisfabib luisfabib merged commit 3aeb3ef into main May 4, 2021
@luisfabib luisfabib deleted the fix/global_weights branch May 4, 2021 08:27
luisfabib added a commit that referenced this pull request May 18, 2021
* deploy_ghpages.yml: add missing dependency (#161)

* Bump VERSION to v0.14.0-dev (#162)

* bump VERSION to 0.14.0-dev
* add Python 3.9 to conda build

* Removed Ctot from second order term in chemicalequalibrium polynomial (#163)

* Fix prescaling of data and vanishing confidence intervals in `snlls` (#166)

* parse_multidatasets: fix bug in parsing so that prescaling works also for single signals

* tests: fix snlls test

* fitregmodel: adapt to changes in parse_multidatasets

* tests: refactor test as relative assertion

* Update README.md (#167)

* deerload: Fix bug when loading BES3T files with entries in manipulation history layer (#164)

* deerload: fix bug when loading BES3T files with entries in manipulation history layer

* add unit tests for deerload

* deerload: fix error in parsing of XGF files

* rename test data files consistently

* fix tests

Co-authored-by: Stefan Stoll <stestoll@users.noreply.github.com>

* Fix behavior of weights in global fitting  (#171)

* fitregmodel: fix behaviour of weights in global fitting

* fitparamodel: add test for global weights behaviour

* lsqcomponents: fix usage of global weights

* fitmultimodel: fix behaviour of weights in global fitting

* fitmultimodel: return list of fitted signals instead of single array, fix scaling of output fit signals and UQ, fix plotting

* fitmultimodel: fix scaling issue in global fitting

* snlls: fix behaviour of weights in global fitting

* fitmodel: add tests to check behaviour of weights in global fitting

* fitmultimodel: fix error when plotting single dataset fits

* fitmultimodel: fix bug in plottin of single dataset fits

* fitmultimodel: fix error in previous commit

* fitmodel: Fix uncertainty quantification when fitting dipolar evolution functions (#172)

* fitmodel: fix errors in uncertainty quantification for dipolar evolution functions

* modify tests that missed the error

* fitmodel: fixed error found by tests

* fitparamodel: fix scale of output fit.model and its uncertainty (#173)

* snlls: allow `extrapenalty` to take linear parameters as input (#175)

* snlls: allow `extrapenalty` to take linear parameters as input

* snlls: fix order of concatenation of Jacobians

* Optimize default weights in global fitting according to the datasets noise levels (#174)

* optimize default weights in global fitting according to the datasets noise levels

* add tests

* fix bug in calculation of default weights for arbitrary scales

* refactor DER_SNR method, add reference

Co-authored-by: Stefan Stoll <stestoll@users.noreply.github.com>

* VERSION: bump to v0.13.1

* CHANGELOG: update for v0.13.1

* CHANGELOG: minor edits

* fix GHA to run on a PR to merge into any branch

* fix GHA to run docs CI on PR to merge into any branch

Co-authored-by: Maxx Tessmer <mhtessmer@gmail.com>
Co-authored-by: Stefan Stoll <stestoll@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Patches something that isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fitregmodel: weights keyword argument has no effect
2 participants