Drop p87482 (AmOpp expense) variable from puf.csv file?

[martinholmer]

The p87482 (American Opportunity qualified expense) variable is in the puf.csv file but is not "aged". It is used in the tax calculations in the AmOppCr function, but when puf == True this calculation has no effect on tax liability because the American Opportunity credit is set to the "aged" value of the p87521 variable, which is also in the puf.csv file.

Given this situation, is there any reason to keep the p87482 variable in the puf.csv file?

Even If the p87482 variable is removed from the puf.csv file, it will remain in the VALID_READ_VARS set (along with the other three student expense variables, p87487, p87492, p87497, which will be moved from the CALCULATED_VARS set). This will allow other datasets to specify the expenses for up to four students and calculate the credit from the expenses when puf == False.

Is there a better way to handle these matters?

@MattHJensen @feenberg @Amy-Xu @GoFroggyRun

[martinholmer]

Issue #742 has been resolved by pull request #747.