Skip to content

Revert "Feature/faster ad tls v4"#1193

Merged
syclik merged 1 commit intodevelopfrom
revert-1171-feature/faster-ad-tls-v4
Apr 2, 2019
Merged

Revert "Feature/faster ad tls v4"#1193
syclik merged 1 commit intodevelopfrom
revert-1171-feature/faster-ad-tls-v4

Conversation

@syclik
Copy link
Member

@syclik syclik commented Apr 1, 2019

Summary

Reverting #1171. This causes problems with Windows.

Tests

N/A

Side Effects

N/A

Checklist

  • Math issue Feature/faster ad tls v4 #1171 (reverting PR 1171)

  • Copyright holder: Generable

    The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
    - Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
    - Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

  • the basic tests are passing

    • unit tests pass (to run, use: ./runTests.py test/unit)
    • header checks pass, (make test-headers)
    • docs build, (make doxygen)
    • code passes the built in C++ standards checks (make cpplint)
  • the code is written in idiomatic C++ and changes are documented in the doxygen

  • the new changes are tested

@wds15
Copy link
Contributor

wds15 commented Apr 1, 2019

Yack... tests fail due to true randomness in the tests. I am restarting them.

Copy link
Contributor

@wds15 wds15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sigh. Back again.

@syclik syclik merged commit 07209fb into develop Apr 2, 2019
syclik added a commit that referenced this pull request Apr 8, 2019
…ster-ad-tls-v4"

This reverts commit 07209fb, reversing
changes made to 7dfc00c.
@syclik syclik deleted the revert-1171-feature/faster-ad-tls-v4 branch April 18, 2019 13:56
wds15 added a commit that referenced this pull request Apr 22, 2019
…ster-ad-tls-v4"

This reverts commit 07209fb, reversing
changes made to 7dfc00c.
syclik pushed a commit that referenced this pull request May 3, 2019
…ster-ad-tls-v4"

This reverts commit 07209fb, reversing
changes made to 7dfc00c.
@serban-nicusor-toptal serban-nicusor-toptal modified the milestone: 2.19.2 Jul 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants