Skip to content

Conversation

@jondo
Copy link
Contributor

@jondo jondo commented Mar 31, 2022

Description

The link to the paper appears correctly as https://arxiv.org/abs/1504.06852 in the docstring of _correlation_cost. The docstring of CorrelationCost however contained a truncated and therefore broken version of this link.

Fixes # (issue) - N/A

Type of change

  • Updated or additional documentation

Checklist:

How Has This Been Tested?

Generating the documentation was not tested.

The link to the paper appears correctly as https://arxiv.org/abs/1504.06852 in the docstring of `_correlation_cost`. The docstring of `CorrelationCost` however contained a truncated and therefore broken version of this link.
@boring-cyborg boring-cyborg bot added the layers label Mar 31, 2022
@bot-of-gabrieldemarmiesse

@failure-to-thrive

You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.

@jondo
Copy link
Contributor Author

jondo commented Mar 31, 2022

@seanpmorgan, you reviewed the original docstring addition. Would you mind having a short look here?

@bhack bhack merged commit 873e4a0 into tensorflow:master Apr 4, 2022
@bhack
Copy link
Contributor

bhack commented Apr 4, 2022

Thanks

@jondo jondo deleted the patch-1 branch April 4, 2022 19:29
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