TheAlgorithms / TheAlgorithms/Python
Feedback on t-SNE script: ideas to improve clarity and usability
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 225k
- Forks
- 51.1k
- Ø Merge
- 1 T. 23 Std.
- Gemergte PRs (30 T.)
- 156
Beschreibung
Feature description
Hi! I tested your t_stochastic_neighbour_embedding.py script on my local setup and it runs well thanks for sharing it!
I have a few suggestions that might improve clarity and usability for others:
Visualization: Consider adding a matplotlib scatter plot to show the t-SNE result. It helps users see the clustering visually.
Reproducibility: Adding random_state=42 to the TSNE() constructor would make the output consistent across runs.
Parameter transparency: Explicitly setting perplexity, learning_rate, and n_iter could help learners understand how t-SNE behaves.
Import error handling: A simple try/except block for sklearn imports could guide users if they haven't installed the package.
Let me know if you'd like help implementing any of these — happy to collaborate!
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Lesen von t_stochastic_neighbour_embedding.py und der aktuellen Verwendung von TSNE. Überprüfe die vorgeschlagenen Änderungen an Visualisierung, Reproduzierbarkeit, Parametern und Importbehandlung und bestätige anschließend, welche Vorschläge im Umfang liegen. Als abgeschlossen gilt die Aufgabe, wenn die ausgewählten Verbesserungen implementiert sind und das Skript weiterhin wie erwartet ausgeführt werden kann.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- matplotlib, python, scikit-learn
- Bereich
- data-visualization, machine-learning
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100