Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented May 4, 2017

  • Add input function for training and testing

Estimator is decoupled from Scikit Learn interface by moving into separate class SKCompat. Arguments x, y and batch_size are only available in the SKCompat class, Estimator will only accept input_fn

  • remove extra comma

* Add input function for training and testing

Estimator is decoupled from Scikit Learn interface by moving into separate class SKCompat. Arguments x, y and batch_size are only available in the SKCompat class, Estimator will only accept input_fn

* remove extra comma
@tensorflow-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@vrv
Copy link

vrv commented May 4, 2017

This is merging a commit from another branch on which the CLA was signed. Testing and merging.

@vrv
Copy link

vrv commented May 4, 2017

@tensorflow-jenkins test this please

@vrv vrv added cla: yes and removed cla: no labels May 4, 2017
@vrv vrv merged commit d0042ed into master May 4, 2017
copybara-service bot pushed a commit that referenced this pull request Feb 22, 2024
Imported from GitHub PR openxla/xla#9650

This adds a new fusion compilation and execution path with corresponding tests.
It will not trigger automatically yet, the mechanism constructing such fusions will be added in another commit.
Also capture of CUDA graphs and wider support of fusions will be added in further commits.
Functionality is intentionally limited to minimal testable for now to make reviewing easier.
Copybara import of the project:

--
9071b7951009e587998bf6f20f69ca8c81d56167 by Ilia Sergachev <isergachev@nvidia.com>:

[XLA:GPU] Add cuDNN GEMM fusions.

This commit adds a new fusion compilation and execution path with corresponding tests.
It will not trigger automatically yet, the mechanism constructing such fusions will be added in another commit.
Also capture of CUDA graphs and wider support of fusions will be added in further commits.

Merging this change closes #9650

PiperOrigin-RevId: 609467714
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants