Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC Use text label instead of emoticon in ML map #30347

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.

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

Merged
merged 3 commits into from
Nov 27, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 9 additions & 5 deletions doc/images/ml_map.README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,12 @@ for exporting the chart are:
- Transparent Background: False
- Appearance: Light

Each node in the chart that contains an estimator should have a link, where the root
directory is at `../../`. Note that after updating or re-exporting the SVG, the links
may be prefixed with e.g. `https://app.diagrams.net/`. Remember to check and remove
them, for instance by replacing all occurrences of `https://app.diagrams.net/../../`
with `../../`.
Note that estimators nodes are clickable and should go to the estimator
documentation. After updating or re-exporting the SVG with draw.io, the links
may be prefixed with e.g. `https://app.diagrams.net/`. Remember to check and
remove them, for instance by replacing all occurrences of
`https://app.diagrams.net/./` with `./` with the following command:

.. prompt:: bash

perl -pi -e 's@https://app.diagrams.net/\./@./@g' doc/images/ml_map.svg
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perl still good for text. (I would have tried to do this with sed. 😅)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have enough space for one of this things in my brain feel free to suggest the equivalent in sed 😉

2 changes: 1 addition & 1 deletion doc/images/ml_map.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
8 changes: 4 additions & 4 deletions doc/machine_learning_map.rst
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,10 @@ data and different problems.

The flowchart below is designed to give users a bit of a rough guide on how to approach
problems with regard to which estimators to try on your data. Click on any estimator in
the chart below to see its documentation. The 😭 emoji is to be read as "if this
estimator does not achieve the desired outcome, then follow the arrow and try the next
one". Use scroll wheel to zoom in and out, and click and drag to pan around. You can
also download the chart: :download:`ml_map.svg <images/ml_map.svg>`.
the chart below to see its documentation. The **Try next** orange arrows are to be read as
"if this estimator does not achieve the desired outcome, then follow the arrow and try
the next one". Use scroll wheel to zoom in and out, and click and drag to pan around.
You can also download the chart: :download:`ml_map.svg <images/ml_map.svg>`.

.. raw:: html

Expand Down