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

snlls: Fix violation of boundaries due to float-point round-off errors #188

Merged
merged 5 commits into from
Jul 2, 2021

Conversation

luisfabib
Copy link
Collaborator

Closes #177

@luisfabib luisfabib added bug Something isn't working bugfix Patches something that isn't working labels Jun 28, 2021
@luisfabib luisfabib added this to the 0.14.0 milestone Jun 28, 2021
@luisfabib luisfabib requested a review from stestoll June 28, 2021 17:40
@luisfabib luisfabib changed the title Jacobian: Fix violation of boundaries to float-point round-off errors Jacobian: Fix violation of boundaries due to float-point round-off errors Jun 28, 2021
deerlab/utils/utils.py Outdated Show resolved Hide resolved
deerlab/utils/utils.py Outdated Show resolved Hide resolved
@luisfabib
Copy link
Collaborator Author

After digging a bit more, the float-point error appears in one of the variables returned by the scipy.optimize.lsq_linear function. I removed the changed to Jacobian and added an explicit control to the results of the linear LSQ results in snlls.

@luisfabib luisfabib changed the title Jacobian: Fix violation of boundaries due to float-point round-off errors snlls: Fix violation of boundaries due to float-point round-off errors Jul 2, 2021
@luisfabib luisfabib requested a review from stestoll July 2, 2021 08:40
@stestoll stestoll merged commit 8fa473f into main Jul 2, 2021
@stestoll stestoll deleted the fix/fitmultimodel_bounds_violation branch July 2, 2021 13:14
luisfabib added a commit that referenced this pull request Jul 2, 2021
…ors (#188)

* Jacobian: add rounding to avoid float-point round-off errors

* Jacobian: use exception handling to avoid rounding in all cases

* snlls: shield linear LSQ results against float-point errors
@luisfabib luisfabib mentioned this pull request Jul 2, 2021
luisfabib added a commit that referenced this pull request Jul 2, 2021
* Enable code analysis by GitHub's CodeQL (#186)

* examples: fix rendering of RST code-blocks in notebooks (#179) (#184)

* Fix call to pipwin executable during PyPI installation (#187)

* `snlls`: Fix violation of boundaries due to float-point round-off errors  (#188)

* docs: fix automated example plots in the models reference (#190)

* VERSION: bump to v0.13.2

* Implement automated PyPI and Anaconda build and publish workflow (#185)

* Update CHANGELOG for v0.13.2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working bugfix Patches something that isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bounds violation error in fitmultimodel
2 participants