Skip to content

Conversation

@jburnim
Copy link
Member

@jburnim jburnim commented Nov 20, 2020

No description provided.

davmre and others added 30 commits November 20, 2020 10:11
Also adds a test to the harness in `structural_time_series_test`.

This fixes a bug where `ssm.copy()` didn't work.

PiperOrigin-RevId: 341714072
former split into separate controller and state classes.

PiperOrigin-RevId: 341846197
…alar function.

Putting this under experimental for now, since it's not clear whether this is compatible with the bijector contract.

PiperOrigin-RevId: 341889649
It's not clear that we *really* need this, but I got most of the way through implementing it before I noticed that `secant_root` already existed. :-) So I figured I might as well check it in, and it does has somewhat different properties.

I tried to mostly mimic the interface of `secant_root`. I think the main practical distinction is that Chandrupatla's method takes a pair of bounds, instead of just an initial position, and is guaranteed to converge if those bounds bracket a root.

PiperOrigin-RevId: 341901341
`NaN` in the extreme cases where `p == 0` or `p == 1`.

PiperOrigin-RevId: 342127106
…r multiple

arg functions. Also support variables and variable tracking via tf.Module.

PiperOrigin-RevId: 342133664
- Use text_fixtures instead of copying.
- Delete redundant tests
- Use `tf.while_loop` instead of Python `for` to iterate over large
  arrays.  This speeds the test up considerably, by saving on graph
  construction time.

PiperOrigin-RevId: 342273733
- Delete redundant tests
- Use `tf.while_loop` instead of Python `for` to iterate over large
  arrays.  This speeds the test up about 2x, by saving on graph
  construction time.

PiperOrigin-RevId: 342699805
they don't defaulted to float64 precision in float64-enabled JAX.

PiperOrigin-RevId: 342720623
…ts in the unit ball.

PiperOrigin-RevId: 342899955
PiperOrigin-RevId: 342909438
…ead of the

former split into separate controller and state classes.

PiperOrigin-RevId: 342956217
The previous re-import logic clobbered too many builtin globals, breaking the
importlib reload functionality.

PiperOrigin-RevId: 343133516
Fix Oryx to point to the internal name jax._src.source_info_util.

PiperOrigin-RevId: 343278153
Use jax.lax.linalg instead.

PiperOrigin-RevId: 343289024
PiperOrigin-RevId: 343322613
@google-cla
Copy link

google-cla bot commented Nov 20, 2020

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

1 similar comment
@googlebot
Copy link

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added the cla: no Declares that the user has not signed CLA label Nov 20, 2020
@jburnim jburnim added cla: yes Declares that the user has signed CLA and removed cla: no Declares that the user has not signed CLA labels Nov 20, 2020
@google-cla
Copy link

google-cla bot commented Nov 20, 2020

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

ℹ️ Googlers: Go here for more info.

1 similar comment
@googlebot
Copy link

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

ℹ️ Googlers: Go here for more info.

@jburnim
Copy link
Member Author

jburnim commented Nov 20, 2020

NOTE: All commits in the PR (besides 1fd985d) are cherry-picked from TFP's master banch.

@jburnim jburnim merged commit ed47dda into tensorflow:r0.12 Nov 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Declares that the user has signed CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.