ENH Add verbose option to SpectralClustering

[sstalley]

Added verbose option to spectral_clustering and SpectralClustering, making issues easier to debug.

Resolves #17969.

Any other comments?

This is my first pull request for sklearn (as well as my first pull request using github),
so any feedback or constructive criticism is appreciated. Thanks!

[TomDLT]

Looks good to me.

You can also add a versionadded tag in the docstring (examples).

[thomasjpfan]

Thank you for the PR @sstalley !

[thomasjpfan]

Please add an entry to the change log at doc/whats_new/v0.24.rst with tag |Enhancement|. Like the other entries there, please reference this pull request with :pr: and credit yourself (and other contributors if applicable) with :user:.

[sstalley]

@thomasjpfan the only other potential author would be you (for the f-string formatting in b9c7af8).
I didn't attribute you for this in the documentation, but can add your name if you like.

Also, I wasn't sure of the best way to resolve the merge conflict. I chose to rebase - but now I see that messes up the discussion history in the PR. Is there a preferred method of resolving merge conflicts for this project?

[thomasjpfan]

I didn't attribute you for this in the documentation, but can add your name if you like.

We normally do not attribute the reviewers, so there is no need to add my name.

Also, I wasn't sure of the best way to resolve the merge conflict. I chose to rebase - but now I see that messes up the discussion history in the PR. Is there a preferred method of resolving merge conflicts for this project?

We prefer to merge with master and handle the conflicts there. When we merge with master, we will square the commits.

For this PR, I pushed to this branch to fix the conflicts and adjust the grammar a little and was about to merge.

Edit: I left a review with the changes.

[sstalley]

We prefer to merge with master and handle the conflicts there. When we merge with master, we will square the commits.
For this PR, I pushed to this branch to fix the conflicts and adjust the grammar a little and was about to merge.

Thanks for the explanation. From now on I will ignore conflicts if the code has already been approved. Sorry for getting in your way this time - I'll try not to let it happen again.

[thomasjpfan]

From now on I will ignore conflicts if the code has already been approved. Sorry for getting in your way this time - I'll try not to let it happen again.

Don't worry about it! It is good to fix the conflicts. In this case, we just so happened to be working on it at the same time. :)

[thomasjpfan]

Thank you for working on this @sstalley !

[sstalley]

Thank you for working on this @sstalley !

Of course. Thanks for your help & review @thomasjpfan !