Skip to content

Conversation

Kalyan1210
Copy link

@Kalyan1210 Kalyan1210 commented Jul 19, 2025

DOC: fix duplicate reference label in SVM narrative (rescue #31765)

Supersedes and closes #31765. Original author seems inactive;
this PR only resolves lint and Sphinx warnings so the doc
improvement can land.

Co-authored-by: originally somebody existed

Copy link

github-actions bot commented Jul 19, 2025

❌ Linting issues

This PR is introducing linting issues. Here's a summary of the issues. Note that you can avoid having linting issues by enabling pre-commit hooks. Instructions to enable them can be found here.

You can see the details of the linting issues under the lint job here


ruff check

ruff detected issues. Please run ruff check --fix --output-format=full locally, fix the remaining issues, and push the changes. Here you can see the detected issues. Note that the installed ruff version is ruff=0.11.7.


benchmarks/bench_tsne_mnist.py:10:1: I001 [*] Import block is un-sorted or un-formatted
   |
 8 |   # SPDX-License-Identifier: BSD-3-Clause
 9 |
10 | / import argparse
11 | | import json
12 | | import os
13 | | import os.path as op
14 | | from time import time
15 | |
16 | | import numpy as np
17 | | from joblib import Memory
18 | | from sklearn.utils._openmp_helpers import _openmp_effective_n_threads
19 | |
20 | | from sklearn.datasets import fetch_openml
21 | | from sklearn.decomposition import PCA
22 | | from sklearn.manifold import TSNE
23 | | from sklearn.neighbors import NearestNeighbors
24 | | from sklearn.utils import check_array
25 | | from sklearn.utils import shuffle as _shuffle
   | |_____________________________________________^ I001
26 |
27 |   LOG_DIR = "mnist_tsne_output"
   |
   = help: Organize imports

Found 1 error.
[*] 1 fixable with the `--fix` option.

Generated for commit: 37f71bd. Link to the linter CI: here

@Kalyan1210 Kalyan1210 force-pushed the doc-rescue-31765 branch 6 times, most recently from 4a427c2 to 0c6a4e0 Compare July 19, 2025 17:00
@StefanieSenger
Copy link
Member

No, original author is not inactive. Closing.

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.

2 participants