search

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

Not showing Whatsapp Data #5

Closed biejay closed 4 years ago

biejay commented 4 years ago

Hey, thanks for your work! Saw your example on Reddit which looked awesome. Unfortunately its not working when i upload a .txt from whatsapp. Maybe due to

  1. language format (it's german and we got Umlaute :D) and

  2. time format (here it is dd/mm/YYYY)?

My format is actually the same like in

Issue visualizing the data #3

but still my visualization looks worse.

Would be awesome if you could update this!

Unbenannt

BryceStevenWilley commented 4 years ago

Thanks for the issue, I'll take a look into this this week. The first thing that would help a lot is if you could post the console log. If you're in Chrome, should press F12 and go to the console tab.

MilanFox commented 4 years ago

Can confirm the bug. Same language and time format.

Console outputs:

Uncaught TypeError: Cannot read property 'concat' of undefined
at math.js:32
at Array.reduce (<anonymous>)
at split_b_k_whatsapp (math.js:26)
at FileReader.<anonymous> (graphs.js:481)
biejay commented 4 years ago

Yes, got slightly the same output like @MilanFox :

Uncaught TypeError: Cannot read property 'concat' of undefined
at math.js:32
at Array.reduce (<anonymous>)
at split_b_k_whatsapp (math.js:26)
at FileReader.<anonymous> (graphs.js:481)
nmatni commented 4 years ago

I changed from dd/mm/YYY to mm/dd/YYYY and still have similar error Uncaught TypeError: Cannot read property 'texts' of undefined at math.js:100 at Array.reduce () at xy_time_of_day (math.js:99) at graphs.js:485

Vidhin05 commented 4 years ago

I changed from dd/mm/YYY to mm/dd/YYYY and I'm getting 

Uncaught TypeError: Cannot read property 'texts' of undefined
    at math.js:101
    at Array.reduce (<anonymous>)
    at xy_time_of_day (math.js:100)
    at graphs.js:485
BryceStevenWilley commented 4 years ago

@biejay, I'd try again, I made some progress on dd/mm/yyyy formats in #2, and I don't think that language differences will be an issue, as the strings being passed around already support emojis, so they're at least Unicode and not ASCII.

BryceStevenWilley commented 4 years ago

@biejay, since you confirmed it's working for you in https://github.com/BryceStevenWilley/visioning_texts/issues/3#issuecomment-576606146, I'm going to close this issue. Everyone else, if you're still running into the issue, follow along at #2!