Skip to content

[FIX] pandas_compat: do not parse column of numbers (object dtype) to datetime#5681

Merged
ajdapretnar merged 1 commit intobiolab:masterfrom
PrimozGodec:pandas-compat-datetime
Nov 10, 2021
Merged

[FIX] pandas_compat: do not parse column of numbers (object dtype) to datetime#5681
ajdapretnar merged 1 commit intobiolab:masterfrom
PrimozGodec:pandas-compat-datetime

Conversation

@PrimozGodec
Copy link
Contributor

Issue

Column with numbers with dtype object was parsed as a TimeVariable

Description of changes

I think it should be parsed as DiscreteVariable or StringVariable since dtype is object

Includes
  • Code changes
  • Tests
  • Documentation

@codecov
Copy link

codecov bot commented Nov 4, 2021

Codecov Report

Merging #5681 (f622c56) into master (f7fe1a8) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #5681      +/-   ##
==========================================
+ Coverage   86.06%   86.07%   +0.01%     
==========================================
  Files         315      315              
  Lines       65995    65999       +4     
==========================================
+ Hits        56796    56808      +12     
+ Misses       9199     9191       -8     

@ajdapretnar ajdapretnar merged commit 2c29c7a into biolab:master Nov 10, 2021
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.

2 participants