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.

Sign up for GitHub

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

Jump to bottom

harmonized_dossier_visualize() report shows wrong total number of variables, missing ID variable #22

Closed
zchenmr opened this issue Oct 18, 2023 · 2 comments
Closed

harmonized_dossier_visualize() report shows wrong total number of variables, missing ID variable #22

zchenmr opened this issue Oct 18, 2023 · 2 comments
Labels
bug Something isn't working to test

Comments

@zchenmr
Copy link

zchenmr commented Oct 18, 2023

R Maelstrom: Rmonize v1.0.0, madshapR v1.0.2, fabR 2.0.0

After running harmonized_dossier_visualize() on the DEMO files, the total number of variables is listed as 12 under "About the dataset - 0.1 Overview". However, the total number of variables (if including the identifier) is actually 13. The identifier variable (adm_unique_id) is also not listed in the table.

image
@GuiFabre GuiFabre added the bug Something isn't working label Oct 19, 2023
@GuiFabre
Copy link
Contributor

good point 👍

image

GuiFabre added a commit that referenced this issue Oct 19, 2023
@GuiFabre
correction of issue :
5e7e6ac 
#22
@zchenmr
Copy link
Author

zchenmr commented Oct 24, 2023

Resolved - the report now displays the identifier variable with the correct number of variables.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working to test
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@GuiFabre @zchenmr