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

[AutoDiff] Enable differentiable programming by default for testing. #31189

Closed
wants to merge 2 commits into from

Conversation

dan-zheng
Copy link
Contributor

The build-script driver argument for differentiable programming is already
enabled by default
, for testing purposes. This makes macOS and Linux CI run
AutoDiff tests.

Now, the CMake flag for differentiable programming is also enabled.
This should make Windows CI run AutoDiff tests as well.

As before, compiler invocations that do not use differentiable programming
features should not be affected.


Let's confirm that AutoDiff tests pass on Windows CI before merging.
AutoDiff test failures on Windows are expected, so we should fix those first.

The build-script driver argument for differentiable programming is already
enabled by default, for testing purposes. This makes macOS and Linux CI run
AutoDiff tests.

Now, the CMake flag for differentiable programming is also enabled by default.
This makes Windows CI run AutoDiff tests as well.

As before, compiler invocations that do not use differentiable programming
features should not be affected.
@dan-zheng dan-zheng requested a review from compnerd April 21, 2020 21:59
@dan-zheng
Copy link
Contributor Author

@swift-ci Please smoke test

@dan-zheng
Copy link
Contributor Author

dan-zheng commented Apr 21, 2020

An AutoDiff test failed on Windows, as expected:

Failing Tests (1):
    Swift(windows-x86_64) :: AutoDiff/validation-test/reabstraction.swift

Fixed in #31201.

@dan-zheng
Copy link
Contributor Author

@swift-ci Please smoke test

@dan-zheng
Copy link
Contributor Author

@swift-ci Please smoke test Windows

@dan-zheng
Copy link
Contributor Author

Re-running to see latest Windows failures.
@swift-ci Please smoke test Windows

@compnerd
Copy link
Member

Right now, I'm seeing these two failures that are yet unresolved:

    Swift(windows-x86_64) :: AutoDiff/validation-test/reabstraction.swift
    Swift(windows-x86_64) :: AutoDiff/validation-test/derivative_registration.swift

Both of these are due to the runtime not finding conformance not being found for the types.

@shahmishal
Copy link
Member

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

  • How to change the base branch: (Link)
  • More detail about the branch update: (Link)

@shahmishal shahmishal closed this Oct 5, 2020
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