Skip to content

Conversation

@PaulWessel
Copy link
Member

The confidence bands for -Er and -Eo were either missing (-Er) or clearly suffered from typos in original manuscript (-Eo) thus giving NaNs. This PR uses the standard propagation of error solution for confidence limits using the expressions for error in slope and intercept from York et al [2004] for the non-weighted cases. The full weighted orthogonal solution may (still) require more testing but left as is for now.

The confidence bands for -Er and -Eo were either missing (-Er) or clearly suffered from typos in original manuscript (-Eo) thus giving NaNs.  This PR uses the standard propagation of error solution for -Er and the equations from G. Borradaile for the orthogonal case [both without data errors only].  The full weighted orthogonal solution may require more testing
@PaulWessel PaulWessel added the backport 6.1 Backport this PR to 6.1 branch label Jul 13, 2020
@PaulWessel PaulWessel merged commit 23beaa5 into master Jul 13, 2020
@PaulWessel PaulWessel deleted the conf-for-orth branch July 13, 2020 18:25
github-actions bot pushed a commit that referenced this pull request Jul 13, 2020
* Calculate confidence band for -Er and -Eo

The confidence bands for -Er and -Eo were either missing (-Er) or clearly suffered from typos in original manuscript (-Eo) thus giving NaNs.  This PR uses the standard propagation of error solution for -Er and the equations from G. Borradaile for the orthogonal case [both without data errors only].  The full weighted orthogonal solution may require more testing

* Update gmtregress.c
PaulWessel added a commit that referenced this pull request Jul 13, 2020
* Calculate confidence band for -Er and -Eo

The confidence bands for -Er and -Eo were either missing (-Er) or clearly suffered from typos in original manuscript (-Eo) thus giving NaNs.  This PR uses the standard propagation of error solution for -Er and the equations from G. Borradaile for the orthogonal case [both without data errors only].  The full weighted orthogonal solution may require more testing

* Update gmtregress.c

Co-authored-by: Paul Wessel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 6.1 Backport this PR to 6.1 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants