Skip to content
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

Fixing broken control-map link in the docs #334

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

robintema
Copy link

Issue #158 mentions that the link is broken. This change fixes the issue.

@@ -53,7 +53,7 @@ box_data = np.array(

Let's create a simple visualization pipeline.
For an explanation of what control-map is doing please check our tutorial at:
[paz/examples/tutorials/controlmap_processor.py](paz/examples/tutorials/controlmap_processor.py)
[paz/examples/tutorials/controlmap_processor.py](https://github.com/oarriaga/paz/blob/master/examples/tutorials/controlmap_processor.py)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Thank you for the PR! Could you please make it a hyperlink? Similar to how it's done on line 5

Copy link
Author

Choose a reason for hiding this comment

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

Hi @proneetsharma !

Sorry, but I don't understand your request. Could you please explain it? I think I did what you're asking. I created an absolute hyperlink like on line 5.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I mean instead of file path you can make with a text, something like this:
For an explanation of what control-map is doing please check our tutorial here.

Copy link
Author

Choose a reason for hiding this comment

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

Done. Please check it again.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks good. Thank you!

Copy link
Author

Choose a reason for hiding this comment

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

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants