-
Notifications
You must be signed in to change notification settings - Fork 368
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
Eliminate PDF from the 6.0 build and packaging #805
Comments
As far as I know, there are still many users who prefer PDF documentations (printed or electronic), especially for people with slow internet connections.
Agree. We can focus on the HTML version, but at least should provide a PDF one.
GMT_Docs.pdf, GMT_Tutorials.pdf, GMT_API.pdf aren't included into the tarballs, right? |
It would be nice that build docs didn't build by default the pdf versions of the figs nor the docs. As is, I always have to remember to delete all pdf's from the build dir before creating the Win installers. |
Currently, |
Yes, that is a good starting point. It probably bogs down the CI to make both sets. |
I remember we have decided to drop PDF documentations in the summit. Is it true? If so, I can make a PR to cleanup the PDF build. |
Yes, we decided to drop the PDF docs from 6.0.0 but retain then in 5.4.x in case we release an update. Not sure if we will though. |
Closed by #1312. |
We have these linting issues because base_plotting.py is too long: ``` pygmt/base_plotting.py:1:0: C0302: Too many lines in module (2047/2000) (too-many-lines) ``` The problem will be fixed in GenericMappingTools#685, but it may take some time. This PR increases the max-module-lines to 3000 to temporarily disable the warning.
PDF documentation is a legacy from the past when we printed stuff. Now it is a limitation since not everything that can be done in HTML translate to PDF. It also adds bulk to the tarballs, etc.
The proposals is to remove all vestiges of PDF from the documentation and the CMake/build scripts. Looking for comments from @GenericMappingTools/core here.
The text was updated successfully, but these errors were encountered: