QuantEcon / meta

For issues and discussion covering more than one repository
0 stars 0 forks source link

[Lectures] Remove titles from Index due to addition of index role #4

Open jstac opened 4 years ago

jstac commented 4 years ago

I haven't looked at the indices of our PDFs in a long time.

Are they OK? Is there any easy way to improve them? All recommendations are appreciated.

Harveyt47 commented 4 years ago

Hey @jstac you mentioned this to me a while ago. Since then I have been reading some of the lectures in the PDFs. I have not seen any issues regarding the indices. But I'll keep looking.

jstac commented 4 years ago

I don't see actually an index, now that I look:

https://python-intro.quantecon.org/_downloads/pdf/quantitative_economics_with_python.pdf

Am I missing something?

Harveyt47 commented 4 years ago

Hey @jstac are you talking about the pdf version of this https://python.quantecon.org/index_toc.html

If so I can see it on page three

jstac commented 4 years ago

That's the table of contents. I'm asking about the index.

mmcky commented 4 years ago

@jstac the index seems to be missing. @Harveyt47 can you research adding index back into LaTeX and we can make sure it is in https://github.com/QuantEcon/lecture-python.theme/blob/master/templates/latex.tpl

mmcky commented 4 years ago

actually it should be in latex_book.tpl.

@jstac index elements are needed in the individual lecture pdf's right?

AakashGfude commented 4 years ago

In case of pdfs, we have actually set all the index files in exclude patterns. So, they are avoided when building pdfs. Like, for example :- https://github.com/QuantEcon/lecture-python-advanced/blob/master/conf.py#L462

Removing it from there, will allow index pages to be built as pdfs

Harveyt47 commented 4 years ago

Thanks @AakashGfude. Would we also have to add \usepackage{imakeidx}, \makeindex and \printindex to latex_bok.tpl?

AakashGfude commented 4 years ago

@Harveyt47 I don't think that will be necessary, as we create tex files for index like any other pages. We can give it a try by removing the entry of index from the config variable and seeing what the pdf output is. If it is behaving weird, then we can try using these packages you mentioned.

Harveyt47 commented 4 years ago

Awesome thanks @AakashGfude I'll give removing index a try and see what happens.

mmcky commented 1 year ago

This issue is now old. There are index items included in all book pdf.

@jstac would you like me to organise a review of that index to see how it is lining up with the current content?

mmcky commented 9 months ago

@jstac the index elements are looking pretty good.

I did notice that many lecture titles are being added.

Screenshot 2024-01-29 at 1 21 10 pm

coming from

# {index}`A Problem that Stumped Milton Friedman <single: A Problem that Stumped Milton Friedman>`

which I think should be removed.

But that was the old issue I found.