add missing currency variables to inflation adjustment

[knaaptime]

• ltdb reader
• ncdb reader

[JennEYoon]

I would like to tackle this, but would need more guidance on what is desired.

Am I to edit "geosnap/data/variables.csv" file? I see under the column "ltdb" various nationalities.

Or is "inflation adjustment" a function or class method defined in a code segment?

Do you have a guide for first time contributors to the geosnap project? Thanks. Hopefully I can add some value.

Footer: Issue #59 -- Add Missing Currency Variables to Inflation Adjustment.

[knaaptime]

awesome, thanks @JennEYoon! we dont have a contributor guide yet, thanks for raising that.

Here are the details on this issue:

we have a function that converts currency variables for inflation. When a user stores a longitudinal dataset like LTDB or NCDB in geosnap's database, the function that reads in the data sends the appropriate columns to the inflation adjuster function. Here is what that looks like for the LTDB reader. Notice that the columns in the list match the currency variables in the ltdb column of variables.csv. What still needs to be done are two things:

1. verify that the columns in inflate_cols on line 494 are all the currency variables in LTDB. This would mean going through each of the rows of variables.csv and looking for those variables that represent currency of some kind and making sure each one is listed in inflate_cols.
2. copy logic similar to lines 494-509 into the store_ncdb function, but making sure that the inflate_cols in this case match the currency variables from the ncdb column in variables.csv

[JennEYoon]

Thanks! I will get on it.

[JennEYoon]

Dear Eli @knaaptime ,
PR ready for your review, Issue #59 Inflation Adjustment.
Sorry it took so long.