Skip to content

Conversation

@APEXCalculus
Copy link
Owner

Mostly the edits are the shifts in figures.
Other edits are made too, changing some wording in places. (Some key ideas, definitions, asides are edited.)

@sean-fitzpatrick
Copy link
Collaborator

I'm having some trouble tracking all of the changes.

  • In the main apex.ptx file, better to not commit the termporary changes to comment out the chapters you're not building.
  • There are a lot of whitespace changes (e.g. in some of the chapter files, some of the <xi:include> lines have been indented for some reason.
  • Later on there are a few things that look like large changes, but I suspect are just changes in indentation. If you can stick to the existing indentation scheme, it'll make it easier for me to follow along
  • In a lot of places you've made a small change (like changing <dl> to <ol>) by commenting out the original, and then adding a copy of it below that has the changes. Unless there is a reason you want to keep the original in a comment, I think it's a lot easier to see what's going on if you just make the chnage.

(Also, the copied versions had a lot of new, extraneous whitespace that made it hard to compare.)

@APEXCalculus
Copy link
Owner Author

Yes, I am sure this pull request is a mess, much of it due to my status as a git noob. I can redo it in a cleaner way if necessary. Problems I foresaw once I committed:

  1. There is too much in this one pull request. I should do one per chapter. Or per section if better.
  2. Yes, apex.ptx is changed and I didn't mean to commit that.
  3. I made changes that I wasn't sure were good, so I commented out old stuff in case we wanted to refer back to the original. I forgot that git handles that kind of issue well; I am too used to doing things on my own.

Is it better if this pull request is cancelled and I:

  • commit in smaller chunks
  • only commit the sections I am editing (and not apex.ptx & chapters, etc.)
  • delete old content and not comment it out, and
  • try to manage whitespace better?

Sorry for the trouble.

@sean-fitzpatrick
Copy link
Collaborator

I can't complain about large pull requests, considering some of the ones I have sent!

Only start over if it's easier for you.

Otherwise, if you can revert the apex.ptx and chapter files, and clean up some of the whitespace, I should be able to work through the rest.

Edits sections that had comments or extraneous whitespace
@APEXCalculus
Copy link
Owner Author

I didn't officially "revert" apex.ptx and chapter files back to the original, but I did restore them to their original state and re-committed them in commit "fixes apex.ptx and chptrs 1 & 2": (7be3a90)

I edited sections to remove most content that I commented out in "fixes sections":
(f2add70)

If I need to adopt a more git-official method, let me know.

One of the most confusing forms of edits is where certain figures needed to be moved before/after certain paragraphs so that they appear on the proper page. So if it looks like I deleted figures, it's because of that.

@sean-fitzpatrick
Copy link
Collaborator

I added a few other changes to the section on basic derivative rules.

The pagebreak element should probably not go inside a paragraph, etc.

A couple of questions for you about changes to the placement of figures, changes to vshifts, etc.:

  1. When you move a figure, will the new location still make sense if the figure is inline? We only put figures in the margin for PDF output. Probably 95% of students (or more) use the HTML version of the book. Very few are interested in PDF. We get only a handful of orders for print copies every year in our bookstore. So I think it's important not to change anything that compromises the HTML output.
  2. Are you using the verison with or without the QR codes for videos? If the new locations overlap with the QR codes, and there isn't room for me to move the QR codes so they fit, I'll have to do the figure placement all over again.

(I could use versioning for this, and have one set of vshifts for videos, and one for no videos, but it would require duplicating half the figures in the book, so I can put one in the video component, with one vshift, and another in the novideo component, with a different vshift.)

@APEXCalculus
Copy link
Owner Author

APEXCalculus commented Aug 1, 2024 via email

@sean-fitzpatrick
Copy link
Collaborator

I think a page break should go between paragraphs, the same as it does in LaTeX.

If the enlarge page command works better elsewhere, that's ok. (In LaTeX I don't think I've ever put it in without line breaks on either side of it.)

Have your students been given the option of the online version of the book? I can understand preferring a hard copy, but I'd be surprised if anyone chose electronic PDF over HTML.

I think people in some places are being pushed to move to HTML to comply with accessibility requirements. But you also get nice perks like interactive features.
(The built in search works really well, too.)

@sean-fitzpatrick
Copy link
Collaborator

I've just figured out that you're not intentionally adding most of the whitespace changes.
Your text editor is set to use tabs for indentation, and your tabs are equivalent to 8 spaces.

Can you change your settings to use 2 spaces for indentation instead of a tab? If you make that change and save your files, it should fix most of the whitespace problems, I think.

If you are using VSCode, use Ctrl+Shift+p to bring up the command pallette. Type 'indent' into the box.
You'll see one line that says "Indent using spaces", which will let you change the setting going forward. (There is also a place you can click on the bottom-right panel to do this.) Another line will say "Convert tabs to spaces". This will make sure there are no tab characters.

@sean-fitzpatrick
Copy link
Collaborator

I made a few changes that were easy to make myself. An earlier comment explains how to fix the whitespace.
There are a few inline comments for things that need your attention.

Other than that, I think this is OK, pending confirmation that the changes are compatible with the HTML output and including video QR codes.

@APEXCalculus
Copy link
Owner Author

I fixed my tab / 2 spaces issue -- I think. We'll see how it looks going forward.

<md>
<mrow>\amp\lzoo{x}{f(x)g(x)} = \lim_{h\to0} \frac{f(x+h)g(x+h)-f(x)g(x)}{h}</mrow>
<mrow>\lzoo{x}{f(x)g(x)} \amp= \lim_{h\to0} \frac{f(x+h)g(x+h)-f(x)g(x)}{h}</mrow>
<mrow>\amp \text{ (now add 0 to the numerator) }</mrow>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Moving the alignment tab here is going to cause a bunch of overfull hboxes.

That will be a problem if this ends up on an odd-numbered page, I think.
And it means that the equations won't fit in HTML: a horizontal slider will be needed.

Copy link
Collaborator

Choose a reason for hiding this comment

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

or maybe even pages, not odd, since there's nothing in the margin there.

@Alex-Jordan
Copy link
Collaborator

About the font/smallcaps/init issue. If there is nothing better to do for PTX at large at this time, you could use the custom XSL that this project already has to change how an init is translated. And just not use small caps.

@sean-fitzpatrick
Copy link
Collaborator

I think I will override the default (which seems questionable, IMO -- I've never seen something like PDF in small caps).

I got the Latte font working on my computer, but I don't want our PDF build to depend on installing some font from source from an 8 year old software repo. Probably Greg doesn't want that, either.

@sean-fitzpatrick
Copy link
Collaborator

@APEXCalculus I will have time this month to return to this. If you would like to connect one day to go through everything, let me know!
I will try to catalogue the changes here, and maybe redo them on a fresh branch -- GitHub is showing some conflicts.

Regarding placement of margin figures in the PDF: why don't we get figures and asides where you want them, and then it'll be my problem to work out how to fit the video QR codes around them.

</li>
</ol>
</p>

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think the change here is to move the figure from the statement of the example to the solution.

Is this being done for page-fitting reasons, or semantic reasons? Semantically, it seems like the figure should be part of the statement. (Note that the statement includes the sentence "The graph in Figure (blah) demonstrates why this is often called a step function.")

We can move the figure into the solution, but then we should either delete this sentence, or move it into the solution. Keep in mind that in HTML (which is the version used by 95% of students), the solution is hidden until they click to reveal it. So they will read the statement of the example, and see a reference to a figure that initially doesn't appear to exist.

Copy link
Owner Author

Choose a reason for hiding this comment

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

I made that change 8+ months ago. I no longer remember why. :)
I was working on figures in the margins, so it would have moved in order to fit on the page better. For sure, it belongs with the question statement, not just the solution. I'd ignore this change and figure out the print version placement later.

Copy link
Collaborator

Choose a reason for hiding this comment

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

My guess was that you wanted the image to move to another page :-)

I'm going through and adding all the non-cosmetic changes on a branch. We can get those in, build the PDF, and then worry about image placement.

<introduction>
<p>
An F-22 aircraft is flying at
An F-22 aircraft is flying at <m><var name="$spd"/>\,\text{mph}</m>
Copy link
Collaborator

Choose a reason for hiding this comment

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

The webwork variable $spd is already a number with units. This will result in a display like 530 mph mph.

APEXCalculus added a commit that referenced this pull request May 12, 2025
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