Skip to content

Conversation

@nelong
Copy link
Collaborator

@nelong nelong commented Jan 15, 2026

Merging in fixes for margins in sidebysides, webwork that is not generating, MK edits to Ch 12

@nelong nelong requested a review from mitchkeller January 15, 2026 19:55
<p>
A tetrahedron is a three-dimensional figure with four faces, each of which is a triangle. A picture of the tetrahedron <m>T</m> with vertices <m>(0,0,0)</m>, <m>(1,0,0)</m>, <m>(0,1,0)</m>, and <m>(0,0,1)</m> is shown in <xref ref="fig-mult_ints-general-PA-tetrahedron">Figure</xref>. If we place one vertex at the origin and let vectors <m>\va</m>, <m>\vb</m>, and <m>\vc</m> be determined by the edges of the tetrahedron that have initial point at the origin, then a formula that tells us the volume <m>V</m> of the tetrahedron is
<md number="yes" xml:id="eq-mult_ints-general-tetrahedron_volume">
<men xml:id="eq-mult_ints-general-tetrahedron_volume">
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be reverted to my version. PreTeXt is deprecating men in favor of md with @number="yes".

<men xml:id="eq-mult_ints-general-tetrahedron_volume">
V = \frac{1}{6} \lvert \va \cdot (\vb \times \vc) \rvert
</md>.
</men>.
Copy link
Contributor

Choose a reason for hiding this comment

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

Need to revert the closing tag as well.

Copy link
Contributor

@mitchkeller mitchkeller left a comment

Choose a reason for hiding this comment

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

@nelong See the one comment I made in a spot where men overwrite a md[number="yes"] change that I had written in Chapter 12. Other than that, it looks fine. (I thought I had spotted an image without a description, but then I realized it was in one of the large commented out chunks. I've been systematically removing those as isolated commits now, but we will have some cleanup to do in that regard after all the editing is done.)

reverting a silly change of mine. Don't remember why I did this
@nelong
Copy link
Collaborator Author

nelong commented Jan 15, 2026 via email

@mitchkeller
Copy link
Contributor

Are you ready for me to merge this, @nelong? I don't have any other things to request changes on, but if you've got more changes you'll be pushing, I can wait to merge until you say it's ready to go.

@nelong
Copy link
Collaborator Author

nelong commented Jan 16, 2026

Your choice. I will be adding image descriptions over the next few days, so we can wait for that or we can merge now and deal with a bigger merge once you are done with Ch 12. You can also keep this open as a way to keep synced with your edits.

@mitchkeller
Copy link
Contributor

I guess you can deploy from your working branch without me merging this, so I'm going to leave open until you feel like you've got through a batch of image descriptions. Tag me when you feel like it would be good to merge, and I will rebase my working branch for Ch 12 off of this before sending in a PR on that.

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.

2 participants