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

E04: add option to import delimited #77

Open
korenmiklos opened this issue Jul 14, 2020 · 1 comment
Open

E04: add option to import delimited #77

korenmiklos opened this issue Jul 14, 2020 · 1 comment
Assignees
Milestone

Comments

@korenmiklos
Copy link
Contributor

  • 'Identifiers sometimes have leading zeros, in which case it may be helpful to store them as string.': This is a very important point and I would stress it even more. Students should understand the structure of the identifier before choosing the type. Otherwise, they may run into problems later where multiple groups are accidentally collapsed in one, e.g. because 01234 is read as 1234 and then cannot be distinguished from 1234. Seen against this background, you could consider to discuss the option asstring for the import statement, otherwise the damage may have already been done.
@andrasvereckei andrasvereckei self-assigned this Jul 23, 2020
@korenmiklos korenmiklos added this to the EEA-2020 milestone Aug 13, 2020
@korenmiklos
Copy link
Contributor Author

@andrasvereckei this seems key, but can you workout an example in time for the workshop? We have space for maybe one callout box.

andrasvereckei added a commit that referenced this issue Aug 18, 2020
Introduce identifiers with leading zero and make a new import delimited challenge. 
Make a E04: add option to `import delimited` #77
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

No branches or pull requests

2 participants