search

danielgp / tableau-hyper-management

Manage importing any CSV file into Tableau-Hyper format (to be used with Tableau Desktop/Server) with minimal configuration (as column detection, content type detection and reinterpretation of content are part of the included logic) with additional script to publish to Tableau Sever as well
GNU Lesser General Public License v3.0
8 stars 2 forks source link

Error creating .hyper file #1

Closed jacobhjkim closed 4 years ago

jacobhjkim commented 4 years ago

Describe the bug A clear and concise description of what the bug is.

To Reproduce Steps to reproduce the behavior:

  1. Example CSV

    user_id,target_level,is_payer,days_for_target_level
abcd,11,false,0.0
abcd,12,false,1.0
abcd,13,false,3.0
abcd,14,false,506.0
abcd,15,false,506.0
abcd,16,false,506.0
abcd,17,false,506.0
xyz,2,false,0.0
xyz,3,false,0.0

  2. python3 tableau-hyper-management/main.py --input-file test.csv --csv-field-separator , --output-file test.hyper

Expected behavior

2020-Feb-20 11:40:53.865085 UTC - Input file is test.csv
2020-Feb-20 11:40:53.865116 UTC - CSV field separator is ,
2020-Feb-20 11:40:53.865129 UTC - Unique values to analyze is limited to 200
2020-Feb-20 11:40:53.865140 UTC - Output file is test.hyper

Traceback (most recent call last): File "tableau-hyper-management/main.py", line 45, in parameters_interpreted) File "/home/ubuntu/tableau-hyper-management/tableau_hyper_management/TypeDetermination.py", line 84, in fn_detect_csv_structure in_prmtrs.verbose) IndexError: list assignment index out of range

Desktop (please complete the following information):

Distributor ID: Ubuntu
Description:    Ubuntu 16.04.6 LTS
Release:    16.04
Codename:   xenial

Tested on c5.4xlarge AWS EC2 instance

Additional context I am not sure why this is giving me a list index out of bounds error. Any help would be appreciated.

danielgp commented 4 years ago

I just found the problem which seems to be a Boolean type that needs to be considered, will issue a fix right away and let you know once release. Thank you very much for raising this out!

danielgp commented 4 years ago

this resolved and new version released, please recheck and let me know

jacobhjkim commented 4 years ago

Thanks! I have tested it and it seems like the issue is resolved.