Documentation for `tf.linalg.set_diag()` is missing return information

[khatchad]

Documentation for `tf.linalg.set_diag()` is missing return information

[khatchad]

Actually, I'm unsure whether these docs are for the correct API. I don't see a call to it anywhere in the examples.

[khatchad]

Ah, they haven't been updated for TF2. They still refer to the old matrix_set_diag API.

[sushreebarsa]

@khatchad You're right, the official TF documentation for tf.linalg.set_diag() doesn't explicitly mention the return value in the descriptiontf.linalg.set_diag. Please refer the master branch for the detailed information.
Thank you!

[github-actions]

This issue is stale because it has been open for 7 days with no activity. It will be closed if no further activity occurs. Thank you.

[github-actions]

This issue was closed because it has been inactive for 7 days since being marked as stale. Please reopen if you'd like to work on this further.

[google-ml-butler]

Are you satisfied with the resolution of your issue?
Yes
No

[sushreebarsa]

@khatchad Could you please let us know if you have referred to the above comment and you could see the return information ?
Thank you!

[khatchad]

@khatchad Could you please let us know if you have referred to the above comment and you could see the return information ? Thank you!

Not sure if I understand the question. It's missing here as well:

tensorflow/tensorflow/python/ops/array_ops.py

Lines 2515 to 2530 in 3825f79

	Args:
	input: A `Tensor` with rank `k + 1`, where `k >= 1`.
	diagonal: A `Tensor` with rank `k`, when `d_lower == d_upper`, or `k + 1`,
	otherwise. `k >= 1`.
	name: A name for the operation (optional).
	k: Diagonal offset(s). Positive value means superdiagonal, 0 refers to the
	main diagonal, and negative value means subdiagonals. `k` can be a single
	integer (for a single diagonal) or a pair of integers specifying the low
	and high ends of a matrix band. `k[0]` must not be larger than `k[1]`.
	align: Some diagonals are shorter than `max_diag_len` and need to be padded.
	`align` is a string specifying how superdiagonals and subdiagonals should
	be aligned, respectively. There are four possible alignments: "RIGHT_LEFT"
	(default), "LEFT_RIGHT", "LEFT_LEFT", and "RIGHT_RIGHT". "RIGHT_LEFT"
	aligns superdiagonals to the right (left-pads the row) and subdiagonals to
	the left (right-pads the row). It is the packing format LAPACK uses.
	cuSPARSE uses "LEFT_RIGHT", which is the opposite alignment.

[tilakrayal]

@khatchad,
Could you please try to contribute with the PR for the respective changes to be made in the mentioned document? Thank you!