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

Include in vignette #156

Closed
suvi-subra opened this issue Jan 15, 2020 · 7 comments · Fixed by #157
Closed

Include in vignette #156

suvi-subra opened this issue Jan 15, 2020 · 7 comments · Fixed by #157
Assignees

Comments

@suvi-subra
Copy link

The following error message could be better explained in the vigenette. Thank you.

"Warning messages:
1: In label_secuTrial.secuTrialdata(d) :
some variable names appear to be duplicated - labels attribute may be longer that 1"

@suvi-subra
Copy link
Author

suvi-subra commented Jan 15, 2020

Another warning:

2: In dates_secuTrial.data.frame(tmp, datevars, timevars, dateformat, :
Not all dates were converted for variable: 'discharge_date' in form: 'nrl08visit_data'
This is likely due to incomplete date entries.

@aghaynes
Copy link
Member

aghaynes commented Jan 15, 2020

Thanks for the feedback.

Another warning:

2: In dates_secuTrial.data.frame(tmp, datevars, timevars, dateformat, : Not all dates were converted for variable: 'discharge_date' in form: 'nrl08visit_data' This is likely due to incomplete date entries.

The bottom of page 6 of the vignette covers this. Is that not sufficient?

@PatrickRWright
Copy link
Collaborator

@aghaynes that was my fault. I forgot that I had entered this in the vignette and asked @suvi-subra to add this to the issue. Maybe we could also add a specific example though?

@suvi-subra
Copy link
Author

Thank you Patrick. I just read it and it seems helpful.

@PatrickRWright
Copy link
Collaborator

@aghaynes we could add a paragraph in the vignette on Errors and Warnings?

@aghaynes
Copy link
Member

@aghaynes we could add a paragraph in the vignette on Errors and Warnings?

That's not a bad idea... you have time?

@PatrickRWright
Copy link
Collaborator

I can do it, yes.

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 a pull request may close this issue.

3 participants