✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

_{Generated for commit: 853a360. Link to the linter CI: here}

See rendered doc

The rendering looks good, though you may need to update the links manually in the generated svg because draw.io seems to automatically resolve relative links. Quote ml_map.README.rst:

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 ../../.

Update: The above instructions seem outdated after we moved the map to under user guide. I think we should now be replacing https://app.diagrams.net/./ with ./.

Thanks, here is the new rendered doc with the fixed links. I check a few links they look good.

I also tweaked the README.

I have noticed there are some mentions in the svg of my repo I created to edit the image with draw.io (https://github.com/lesteve/drawio-playground) but I guess it's OK enough as long as the graphical rendering of the svg is fine 🤷:

If I search in the svg, I see for example:

resource="https://app.diagrams.net/#Hlesteve%2Fdrawio-playground%2Fmain%2Fml_map.svg#%7B%22pageId%22%3A%22prGmxGi5H6ogpCY3go2q%22%7D">