replace deprecated function abs by fabs

[migueldvb]

Replace abs by fabs in lasso example.

[stan-buildbot]

Can one of the admins verify this patch?

[andrewgelman]

“fabs”???

On Oct 28, 2015, at 4:45 PM, Miguel de Val-Borro notifications@github.com wrote:

Replace abs by fabs in lasso example.

You can view, comment on, or merge this pull request online at:

#1649 #1649
Commit Summary

replace deprecated function abs by fabs
File Changes

M src/docs/stan-reference/advanced.tex https://github.com/stan-dev/stan/pull/1649/files#diff-0 (2)
Patch Links:

https://github.com/stan-dev/stan/pull/1649.patch https://github.com/stan-dev/stan/pull/1649.patch
https://github.com/stan-dev/stan/pull/1649.diff https://github.com/stan-dev/stan/pull/1649.diff
—
Reply to this email directly or view it on GitHub #1649.

[bob-carpenter]

See the manual index for definitions and signatures
for abs() and fabs() and the corresponding min/max and fmin/fmax.

See old mail threads for discussion on why it's like that.

But before you tell us again that it doesn't have to be that way
and that it's "tacky", see:

https://github.com/stan-dev/stan/issues/1255

It's scheduled for Stan 3 because it's going to break backward
compatibility (unless we keep fabs and deprecate it).

This low-level maintenance coding never gets done because
nobody wants to do it.

• Bob

On Oct 28, 2015, at 4:46 PM, Andrew Gelman notifications@github.com wrote:

“fabs”???

On Oct 28, 2015, at 4:45 PM, Miguel de Val-Borro notifications@github.com wrote:

Replace abs by fabs in lasso example.

You can view, comment on, or merge this pull request online at:

#1649 #1649
Commit Summary

replace deprecated function abs by fabs
File Changes

M src/docs/stan-reference/advanced.tex https://github.com/stan-dev/stan/pull/1649/files#diff-0 (2)
Patch Links:

https://github.com/stan-dev/stan/pull/1649.patch https://github.com/stan-dev/stan/pull/1649.patch
https://github.com/stan-dev/stan/pull/1649.diff https://github.com/stan-dev/stan/pull/1649.diff
—
Reply to this email directly or view it on GitHub #1649.

—
Reply to this email directly or view it on GitHub.

[andrewgelman]

That’s fine. If it’s in the manual, I can deal with it.
A

On Oct 28, 2015, at 5:09 PM, Bob Carpenter notifications@github.com wrote:

See the manual index for definitions and signatures
for abs() and fabs() and the corresponding min/max and fmin/fmax.

See old mail threads for discussion on why it's like that.

But before you tell us again that it doesn't have to be that way
and that it's "tacky", see:

https://github.com/stan-dev/stan/issues/1255

It's scheduled for Stan 3 because it's going to break backward
compatibility (unless we keep fabs and deprecate it).

This low-level maintenance coding never gets done because
nobody wants to do it.

• Bob

On Oct 28, 2015, at 4:46 PM, Andrew Gelman notifications@github.com wrote:

“fabs”???

On Oct 28, 2015, at 4:45 PM, Miguel de Val-Borro notifications@github.com wrote:

Replace abs by fabs in lasso example.

You can view, comment on, or merge this pull request online at:

#1649 #1649
Commit Summary

replace deprecated function abs by fabs
File Changes

M src/docs/stan-reference/advanced.tex https://github.com/stan-dev/stan/pull/1649/files#diff-0 (2)
Patch Links:

https://github.com/stan-dev/stan/pull/1649.patch https://github.com/stan-dev/stan/pull/1649.patch
https://github.com/stan-dev/stan/pull/1649.diff https://github.com/stan-dev/stan/pull/1649.diff
—
Reply to this email directly or view it on GitHub #1649.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub #1649 (comment).

[syclik]

This looks fine.