Skip to content

DOC Broken link: Working with Text Data Tutorial Setup #29347

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
jaderrrr opened this issue Jun 24, 2024 · 3 comments
Closed

DOC Broken link: Working with Text Data Tutorial Setup #29347

jaderrrr opened this issue Jun 24, 2024 · 3 comments
Labels
Documentation

Comments

@jaderrrr
Copy link

Describe the issue linked to the documentation

In the setup section of the Working With Text Data tutorial, it says:

The source of this tutorial can be found within your scikit-learn folder:
scikit-learn/doc/tutorial/text_analytics/
The source can also be found on Github.

However, the doc/tutorial folder has been deleted (#29104), so both of these instructions are broken.

Suggest a potential alternative/fix

Either put the required data back in the repository somewhere, or adjust the tutorial so that there is no longer any references to this setup. The only parts of the tutorial which really require the setup are the exercises, which refer to running on the dataset created from scikit-learn/doc/tutorial/text_analytics/data/languages/fetch_data.py.
@jaderrrr jaderrrr added Documentation Needs Triage Issue requires triage labels Jun 24, 2024
@jeremiedbb
Copy link
Member

Hi @jaderrrr, The data has been removed in the development version of scikit-learn (1.6.0dev0) but you're following the documention of the stable version (1.5.0). This tutorial has actually been removed as well in the doc of the dev version (https://scikit-learn.org/dev/). If you still want to follow this tutorial I would suggest git checkout 1.5.0 or retrieve the data on github, in the 1.5.X branch (https://github.com/scikit-learn/scikit-learn/tree/1.5.X/doc/tutorial). Note that these tutorial haven't been maintainted in a while so they might use an outdated API and not work anymore.

@jeremiedbb jeremiedbb removed the Needs Triage Issue requires triage label Jun 25, 2024
@jaderrrr
Copy link
Author

Thanks @jeremiedbb, I will do that :)

@glemaitre
Copy link
Member

I'm wondering if we should do a redirection.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@glemaitre @jeremiedbb @jaderrrr