missing separator in get_view_data_dataframe

divinorum-webb / tableau-api-lib

An API library that allows developers to call on the methods listed in Tableau's REST API documentation.

MIT License

96 stars 34 forks source link

missing separator in get_view_data_dataframe #74

Closed Ventrilok closed 2 years ago

Ventrilok commented 2 years ago

Hello,

First, thanks for this lib, so useful for me and my team !

We recently identify a small issue with the get_view_data_dataframe that raised a tokenizing data error. After a quick investigation we noticed that it was necessary to explicitly specify the separator: view_df = pd.read_csv(StringIO(view_data.text), sep=";")

It would be very ulseful to add an additional argument in the get_view_data_dataframe method to specify the separator.

Thanks again for this great lib!

divinorum-webb commented 2 years ago

Hey @Ventrilok great feedback, thanks! Go ahead and run pip install -U tableau-api-lib and the library will now support the user-defined sep param for that function.