chrislgarry / Apollo-11

Original Apollo 11 Guidance Computer (AGC) source code for the command and lunar modules.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Proof POWERED_FLIGHT_SUBROUTINES

wopian opened this issue · comments

Proof read transcribed code of POWERED_FLIGHT_SUBROUTINES against scans

Lines: 447
Page: 1259—1267 (8)

Reduced quality scans can be found here

I'm going to attempt this one if no one else is.

commented

@serialhex how's it going?
if youre tired of proofreading, ill be happy to pick up where you left off. :)

commented

I just started looking through it, and noticed on page 1262, that the word TRANSFORMED is hyphenated in the original, and not in the code here.
I assume that is an acceptable difference?
since it's a comment anyway.

Screenshot from 2020-10-09 14-18-17

Hey, sorry I didn't get back to you @tcuc I don't have reliable internet access, so I need to plan my updates accordingly. As far as I know & understand, the code should be exactly like the transcript, including misspellings. At least that's how I've been working on it.

I should have this done by the weekend sometime, maybe Monday.

I just started looking through it, and noticed on page 1262, that the word TRANSFORMED is hyphenated in the original, and not in the code here.

This should be corrected to match the scans.

since it's a comment anyway.

The proofing is focused solely on the comments (for the most part)

Is this still open? Is there something I can help with?

Is this still open? Is there something I can help with?

@serialhex is already working on this

@wopian Question: I'm almost done with my transcription, but I just noticed that a lot of the code uses spaces instead of (or with) tabs in some places. Such as this:

# lines 77-80
CDUTRIGS	CA	CDUX
		TS	CDUSPOT +4	# the whitespace is a space
		CA	CDUY
		TS	CDUSPOT

and

# lines 88-93, removed comments
TR*GL**P	MASK	SIX
		TS	TEM3
		INDEX	TEM3
		CA	CDUSPOT
		DXCH	MPAC
		DXCH	VBUF 	+4	# Whitespace is a space and then a tab

There's even this:

# lines 434-437
		DXCH	MPAC 	+3	# space-tab between "MPAC" and "+3"
		DDOUBL
		INDEX 	TEM1		# space-tab between "INDEX" and "TEM1"
		DXCH	10

I don't know enough about AGC assembly to know if this is significant, but I doubt it is. Should I remove these spaces? It looks like it wouldn't mess up the alignment of the code at all, even in the first example the tab would put the +4 in the same column.

Everything else is done, I'm just curious about this whitespace issue. Thank you!

I don't know enough about AGC assembly to know if this is significant, but I doubt it is. Should I remove these spaces? It looks like it wouldn't mess up the alignment of the code at all, even in the first example the tab would put the +4 in the same column.

Should be tabs only for the (uncommented) code itself