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

added note about text being formatted as dates #231

Merged
merged 1 commit into from
May 3, 2018

Conversation

tracykteal
Copy link
Contributor

Addresses #5, adding the information that spreadsheet programs can mis-interpret text as dates.

@hoytpr
Copy link
Contributor

hoytpr commented Apr 13, 2018

I think we may need something a little more definitive, like an example. I know we don't want to add time to these lessons, but when working with 50,000 rows, this can be easily missed. The easiest example is to simply type a gene name like "SEP1" into a cell. It's automatically converted to "01-Sep" and in the formula bar it will say "09/01/2018".

If you try to copy the "01-Sep" and paste it as text, you get "43344". So Excel permanently changes your gene name to a date if typed into a native cell. Your best bet is to "preformat" the cells/columns as "TEXT" before you paste. Instinctually one wants to change the cell-type of pasted, date-formatted "genename" to "TEXT" but that will just just convert the date to a number, stored as text. Gets ugly fast.

I could write up a short example for "further reading". That way the class wouldn't be slowed.
This change looks okay for now. Until I get a new page together, this should probably be merged (LGTM).

@hoytpr hoytpr merged commit e01f028 into datacarpentry:gh-pages May 3, 2018
fmichonneau pushed a commit that referenced this pull request Jun 19, 2018
bin/util.py: fix error handling for Python 2
zkamvar pushed a commit that referenced this pull request May 3, 2023
added note about text being formatted as dates
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 this pull request may close these issues.

None yet

2 participants