Skip to content

Conversation

@vishwakftw
Copy link
Contributor

This completes feature request #6233 .
Tests have been added as well.

cc: @apaszke @soumith

virtual Tensor log2(const Tensor & self) const;
virtual Tensor & lgamma_out(Tensor & result, const Tensor & self) const;
virtual Tensor lgamma(const Tensor & self) const;
virtual Tensor & lgamma_(Tensor & self) const;

This comment was marked as off-topic.

@ezyang
Copy link
Contributor

ezyang commented Apr 4, 2018

Windows failure is legit.

17:47:05 src\TH\THGeneral.h(151): error C2169: 'log10': intrinsic function, cannot be defined

ezyang
ezyang previously requested changes Apr 4, 2018
Copy link
Contributor

@ezyang ezyang left a comment

Choose a reason for hiding this comment

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

Windows

@vishwakftw
Copy link
Contributor Author

The Windows build has been fixed. The only test failing is onnx-fb-universe.

@vishwakftw vishwakftw changed the title [pytorch] Implement log2 and log10 in PyTorch [ready] Implement log2 and log10 in PyTorch Apr 5, 2018
@ezyang ezyang merged commit 0aa3578 into pytorch:master Apr 5, 2018
ezyang added a commit that referenced this pull request Apr 5, 2018
soumith pushed a commit that referenced this pull request Apr 5, 2018
…error" (#6322)

* Revert "Add __constants__ to Script modules (#6092)"

This reverts commit 5ab30ee.

* Revert "[ready] Implement log2 and log10 in PyTorch (#6272)"

This reverts commit 0aa3578.

* Revert "Use reshape({-1}) (#6281)"

This reverts commit 8ae67a4.

* Revert "Move instruction set specific code to anonymous namespace (#6314)"

This reverts commit 6953c1b.

* Revert "[auto] Update onnx to 54be8fa - Use cmake3 if it's available (#718) onnx/onnx@54be8fa"

This reverts commit d33ec12.

* Revert "default build with MKL for desktop (#6266)"

This reverts commit 5dcf707.

* Revert "Increase # of runs for CPU perf test, and increase margin of error (#6302)"

This reverts commit 9d1a660.
@vishwakftw vishwakftw deleted the log2-log10 branch May 28, 2018 13:36
wuhuikx pushed a commit to wuhuikx/pytorch that referenced this pull request Jan 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants