Apply dictionary 2023 09 18

[epi-clz]

Hi girls! Thank you so much for reviewing. I have made changes in the following functions: openDictionary() applyDictionary() compareDataset() I have noted that with ## PR_CLZ and ##END_PR_CLZ Let me know if anything is unclear or you need help

[epi-jhd]

Comments after testing the openDictionary() function (all other changes look great! )

• Do any dictionary functions interact with the comments column? -> if not, suggest ignoring this column if present
• Extra sheets in the dictionary causes a warning -> suggest ignoring extra sheets if the main three are present (Sometimes it is useful to have extra reference sheets in dictionaries -not to use in R, but to keep useful information all in one place)
• When one of the sheets is totally empty, there is no warning or explanation -> suggest creating the columns and inform that have done so.

[epi-gde]

I agree with Jenny, no warning is necessary if we have extra sheet or extra column as long we don't use them for dictionary

[epi-clz]

Hi all. In order go through Jenny's suggestions I'd like to share the workflow of the function now:

1. Check if the essential sheets are found in the excel. Otherwise print warning.
2. If Ok, check if all sheet are not completely blank. Otherwise print warning.
3. If Ok, check if any of the sheets lacks of any essential column. Otherwise print warning and creates automatically the cols as blank.

If everything Ok -> runs the function with no problems If necessary, adds cols missing in the current user dictionary as blank, warning the user with a message. @epi-jhd @epi-gde @loremerdrignac

[epi-jhd]

Had a check and it's working like a dream :)

[epi-mml]

It works very well , well done Cris :)! I just have an idea/suggestion: do you think it can be useful to precise the missing sheet ? (For example, if the "actions" sheet doesn't exist, print "Sheet in XXX : actions sheets is missing".