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

Add 67-FM1-38 #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add 67-FM1-38 #37

wants to merge 1 commit into from

Conversation

spencerkee
Copy link
Contributor

Add 67-FM1-38 from pg. 19 of http://web.mit.edu/digitalapollo/Documents/Chapter7/tindallgrams.pdf

I'm not sure of the policy on this but in the pdf there's the line computer program formulation will not handle that, The current formulation. The character between that and The looks like a comma, but should probably be a period. I set it to a period in my change but I can swap it if needed.

Also should I add Jekyll font matter for the To: See list line?

@seanredmond
Copy link
Owner

Since this PDF is showing some kind of OCR, there's not way to tell if that comma is original or not so changing it to a period is a pretty minor edit.

For consistency don't add the To: line. Most scans don't include the list so there is nothing to refer to.

I'll check this over as soon as I can and merge it if I don't see any problems

@@ -0,0 +1,16 @@
---
layout: tindallgram
date: May 17 1997
Copy link
Owner

Choose a reason for hiding this comment

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

Can you correct this date?

program formulation will not handle that. The current formulation
is designed to handle a minimum L/D value of .30 and, although they do
not know how far this limit can be extended, they feel certain major
rework will be required for .25. This is no a new discovery. Our
Copy link
Owner

Choose a reason for hiding this comment

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

"not a new discovery" ?

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