search

datacarpentry / spreadsheets-socialsci

Data Organization in Spreadsheets for Social Scientists
http://datacarpentry.github.io/spreadsheets-socialsci/
Other
21 stars 68 forks source link

Quality assurance: Excel refuse to validate data on CSV file. #89

Open Carreau opened 5 years ago

Carreau commented 5 years ago

the Quality assurance section suggest to open the clean CSV and apply Validation Rules. Excel will refuse to do that until the CSV is re-saved as xlsx.

As we don't really use CSV until later, could/should(?) the default file be an XLSX ?

chris-prener commented 5 years ago

My preference would be to continue to use .csv files but include language that it must be re-saved as this point to run the validation process. What version of Excel are you using, @Carreau, and what OS?

Carreau commented 5 years ago

Excel 16.25 (latest) and mac 10.14 (latest), we are teaching this lesson in a few minutes, so will see if this is a local issue and report back.

chris-prener commented 5 years ago

Great, thanks @Carreau!

Carreau commented 5 years ago

Well, that literally depend on Excel mood. My computer now is happy, but one of the attendee (out of 12) had the issue during the workshop (on windows this time). It likely a case of "can't print on Tuesday"... so I guess we can close, or maybe just have an advisory for instructors this might happen

chris-prener commented 5 years ago

Thanks for the update, @Carreau - curious behavior indeed. I think adding something to the instructor notes for this lesson is the best strategy. Could you perhaps try to get the behavior to occur again and take some screenshots? That would be really helpful for us as we update the instructor notes accordingly!

Carreau commented 5 years ago

Screenshot won't help. Clicking the button to bring the validation dialog just does nothing. The dialog refuses to pop up.

The button is not grayed or anything, it just does not trigger the dialog to pop up.

The rest of the UI works fine.

On Fri, May 24, 2019, 12:35 Christopher Prener notifications@github.com wrote:

Thanks for the update, @Carreau https://github.com/Carreau - curious behavior indeed. I think adding something to the instructor notes for this lesson is the best strategy. Could you perhaps try to get the behavior to occur again and take some screenshots? That would be really helpful for us as we update the instructor notes accordingly!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/datacarpentry/spreadsheets-socialsci/issues/89?email_source=notifications&email_token=AACR5T26IRMELFYGUDPOTXLPXA7Q7A5CNFSM4HOOL2LKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWGMM3Q#issuecomment-495765102, or mute the thread https://github.com/notifications/unsubscribe-auth/AACR5T56VLPOF5QPEWIFCHDPXA7Q7ANCNFSM4HOOL2LA .

chris-prener commented 5 years ago

ok great - thanks for the details. I'll circle back around when I've made the change!