ddalthorp / GenEst

R package development for a generalized mortality estimator
Other
6 stars 6 forks source link

prepopulation of variables #471

Closed mhuso closed 6 years ago

mhuso commented 6 years ago

I did not enter "DateFound" How did GenEst know to populate with this variable? Is that desirable?

image

mhuso commented 6 years ago

Using CO_cleared

ddalthorp commented 6 years ago

That is the only plausible "date found" column in your data set.

mhuso commented 6 years ago

What happens if there are two? No auto=completion?

< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: ><

< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: >< :: ><

Manuela Huso Research Statistician USGS Forest and Rangeland Ecosystem Science Center Forest Sciences Lab, Rm 156 3200 SW Jefferson Way Corvallis, OR 97331 ph: 541-750-0948 cell: 541-760-8520 mhuso@usgs.gov

http://usgs.gov/fresc

From: ddalthorp notifications@github.com Sent: Wednesday, August 22, 2018 10:00 AM To: ddalthorp/GenEst GenEst@noreply.github.com Cc: Manuela Huso mhuso@usgs.gov; Author author@noreply.github.com Subject: [EXTERNAL] Re: [ddalthorp/GenEst] prepopulation of variables (#471)

That is the only plausible column in your data set.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ddalthorp/GenEst/issues/471#issuecomment-415103816, or mute the thread https://github.com/notifications/unsubscribe-auth/AcwzA2kBKMzYdjErnTTO-p6Rn69zxu5Jks5uTY4OgaJpZM4WIC8b .[image: Image removed by sender.]

ddalthorp commented 6 years ago

I don't know....If there are two possibilities, I presume there will be two choices in the drop down menu.

On the command line side, you would be entering args in the estM function, including a dateFound var. If dateFound = NULL, the function will search for plausible candidates. If exactly one is found, that becomes the dateFound column; otherwise, an error is thrown and the user is forced to provide one.

juniperlsimonis commented 6 years ago

correct. and now if there are multiple plausible columns it will not select one by default