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

feat: enhance and fix documentation #1310

Conversation

barendgehrels
Copy link
Collaborator

@barendgehrels barendgehrels commented Sep 22, 2024

The registration macros were not clear enough. They were not yet stating that longitude should come first, and then latitude.

Adding that, I encountered that the documentation procedure was broken on my Mac. This was caused in the end by this
commit: boostorg/boostbook@116c7b0 - so I wonder why it can run in the main branches. See also boostorg/boostbook#17

Anyway, after fixing that locally I can generate the same documentation again. There were quite some warnings (by doxygen, or doxygen to qbk) which are fixed. Also some other unclarities are enhanced.

Some pages were not used for already 10 years and are removed. This was in doxygen - but we use quickbook now.

Documentation looks now like (some shots):

image image

@barendgehrels
Copy link
Collaborator Author

BTW, it's quite confusing that b2 calls make_qbk.py and, vice versa, make_qbk.py can optionally call b2 (in release mode).
Which one is effectively used in release documentation?


echo "=== Resulting contents of generated 'adapted.html', there should be sections"
cat html/geometry/reference/adapted.html | grep section

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@Lastique also for your information ('ll refer to it from elsewhere)

@barendgehrels barendgehrels force-pushed the feat/fix-and-enhance-documentation branch from 694d6de to 23c1455 Compare September 25, 2024 07:19
Copy link
Member

@vissarion vissarion left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks!

@barendgehrels barendgehrels merged commit 218bc06 into boostorg:develop Sep 27, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants