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

Updated narf_keypoint_extraction.rst #1777

Conversation

rahulchhabra07
Copy link
Contributor

Added hyperlink to a tutorial referred.

Added hyperlink to a tutorial referred.
@VictorLamoine
Copy link
Contributor

👍

@rahulchhabra07
Copy link
Contributor Author

Hey,

The merging of this pull request seems to have failed.
Could you please let me know the possible reasons and any changes that I should make.

Thanks.

@SergioRAgostinho
Copy link
Member

@rahulchhabra07 Nothing related to the changes you did.

@SergioRAgostinho SergioRAgostinho merged commit eab590a into PointCloudLibrary:master Dec 11, 2016
@taketwo
Copy link
Member

taketwo commented Dec 13, 2016

I think we should use RST commands instead of explicit urls:

:ref:`range_image_visualization`

@rahulchhabra07
Copy link
Contributor Author

Yeah, that sounds better.

Will make another commit for that.

Let me know if you have any more suggestions. I am planning to add references for the entire documentation wherever necessary.

Thanks.

@taketwo
Copy link
Member

taketwo commented Dec 13, 2016

That would be a great contribution, thanks in advance!

@rahulchhabra07 rahulchhabra07 deleted the rahulchhabra07-patch-1 branch December 16, 2016 12:40
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.

4 participants