BryceStevenWilley / visioning_texts

A D3 project that locally visualizes your messages from Signal or Whatsapp
GNU General Public License v3.0
37 stars 6 forks source link

Extra conversation participants #2

Closed filipedematos closed 4 years ago

filipedematos commented 4 years ago

Hi Bryce! Congratulations on your tool, it really looks awesome! It was shared in r/dataisbeutiful

Tried it myself but got this instead:

Screenshot 2020-01-20 at 16 13 20

The file is from a whatsapp backup, not english though, portuguese.

Thank you in advance!

samsamduvent commented 4 years ago

Hello, i come from reddit as well and have the same problem. followed the instruction and it didn't work. Whatsapp in english. Tried it locally and on the web in chrome and safari.

BryceStevenWilley commented 4 years ago

So, when you say WhatsApp backup, do you mean the full WhatsApp backup file of all conversations? I only had time to handle 2 people conversations, and it looks like you uploaded a file with a lot of different people in it.

samsamduvent commented 4 years ago

Just 1-1 conversion. I exported the chat, unzipped the file and uploaded it

Vidhin05 commented 4 years ago

@BryceStevenWilley This is a typical line in my whatsapp file, [21/12/18, 22:17:43] NAME: ‎TEXT What does your's look like?

Vidhin05 commented 4 years ago

This issue seems connected to https://github.com/BryceStevenWilley/visioning_texts/issues/6

BryceStevenWilley commented 4 years ago

That's likely the issue: my WhatsApp file looks like: 5/21/19, 15:39 - NAME: TEXT

No braces or anything. Thanks for the help @Vidhin05

BryceStevenWilley commented 4 years ago

I just pushed some changes that should handle [ WhatsApp output. @Vidhin05, @filipedematos, @samsamduvent, let me know if it fixed anything for you. If it didn't, could you look into the console log and post any errors?

slevin1991 commented 4 years ago

Same problem as OP but now it's blank when I run my file.

Typical line: 5/21/19, 15:39 - NAME: TEXT

Console info: math.js:68 Uncaught TypeError: Cannot read property 'concat' of undefined at math.js:68 at Array.reduce () at split_b_k_whatsapp (math.js:62) at FileReader. (graphs.js:488)

BryceStevenWilley commented 4 years ago

Thanks @slevin1991, I made a dumb mistake and broke some other fixes. Sholud be fixed by https://github.com/BryceStevenWilley/visioning_texts/commit/73734339bbcac6fb90653d064983c26c2f519a75.

Vidhin05 commented 4 years ago

@BryceStevenWilley https://github.com/BryceStevenWilley/visioning_texts/commit/799f6e84709af7696ff22de0c3b60574a5272c95 Fixed everything for me! Thank you

slevin1991 commented 4 years ago

EDIT: It's because my .txt file is too big. It's at 30mb, just had to wait for it to run.

It still shows up as blank for me :(

Uncaught TypeError: Cannot read property 'concat' of undefined at math.js:109 at Array.reduce () at split_b_k_whatsapp (math.js:103) at FileReader. (graphs.js:488)

samsamduvent commented 4 years ago

@BryceStevenWilley just tested the newest version and got this from the console:

Btw a sample line of the chat file: [9/20/19, 09:31:09]

math.js:109 Uncaught TypeError: Cannot read property 'concat' of undefined at math.js:109 at Array.reduce (<anonymous>) at split_b_k_whatsapp (math.js:103) at FileReader.<anonymous> (graphs.js:488)

BryceStevenWilley commented 4 years ago

@samsamduvent, (also everyone else still having problems in this thread), I've fixed much more of the date parsing issues, and have confirmed it with some tests. Check it out again, and let me know how it goes

samsamduvent commented 4 years ago

@BryceStevenWilley great it works! thanks a lot. Still having #8 though.

BryceStevenWilley commented 4 years ago

Awesome, thanks! I'll close this issue for now, and will try to make some progress on #8 this week.