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

Installation page added #1694

Open
wants to merge 1 commit into
base: rtd-docs
Choose a base branch
from
Open

Conversation

ayushya19
Copy link

I have taken content from the gtsam website and from the doxygen docs to insert content on the subtopics Importing "GTSAM in your projects" and "Installing"

@dellaert
Copy link
Member

Thanks !!! I'd like to check out your branch and check it out.
It's been a while since I looked at rtd, though. What are the steps to render/view ?

@varunagrawal
Copy link
Collaborator

varunagrawal commented Jan 3, 2024

@dellaert the steps involved are

git checkout rtd-docs
cd docs
make html
# For Linux
xdg-open build/html/index.html
# For Mac
open build/html/index.html

@ayushya19 I think you have committed the wrong files. Your PR includes the build directory which is created when we run make html. You probably intended to update and commit the docs/source/Installing.rst file.

@varunagrawal
Copy link
Collaborator

@ayushya19 are you still intent on completing this PR?

@ayushya19
Copy link
Author

Hey @varunagrawal ,

Sorry for the late responses. I have the files ready.

Just have been a lil caught up. I'll push the changes as soonas I get a chance.

@varunagrawal
Copy link
Collaborator

@ayushya19 any updates?

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