search

djsudduth / keep-it-markdown

Convert Google Keep notes dynamically to markdown for Obsidian, Logseq, Joplin and Notion using the unofficial Keep API. Also, import simple markdown notes back into Google Keep.
Apache License 2.0
493 stars 32 forks source link

Language error in keep_query_convert() on Win10 #82

Closed ndilekli closed 2 months ago

ndilekli commented 3 months ago

Ran the code with the prompt "python kim.py -b --all" to process all the notes but got the message:

Error in keep_query_convert() Could not excute KIM

after a few notes. Not sure what the nature of the error, but if the code skipped the file causing the issue (while printing the name of the problem file), I could add those manually later.

Also previously I had the error/warning:

'Keep.resume' has been renamed to 'Keep.authenticate'. Please update your code

So I updated the kim.py accordingly (replacing 3 instances of resume with authenticate)

error

djsudduth commented 3 months ago

@ndilekli Thanks for submitting the issue. Is there any way to narrow down what note might have caused the error? It would be great to see what might have been the problem.

In the meantime I'll fix the other issues and do the skip over problem notes.

ndilekli commented 3 months ago

Strangest thing. I originally tried it on a Win10 machine. Just tried it on a Win11 machine, and everything went smoothly, converted over 600 notes. One other thing I did was to add titles to the top few notes in order to identify which one was causing the problem. But never had the error. Thanks for everything :)

If you want I will close the Issue.

Edit: Can confirm it, I am having the issue on the Win10 machine. Installed the latest version of Python on both. I suspect the issue originates due to international characters (Turkish to be exact), but I am not 100% sure.

djsudduth commented 3 months ago

Interesting. I've tested international characters - Mandarin and Russian but not Turkish. Could it be a codepage issue in Windows 10 vs 11?

ndilekli commented 3 months ago

I don't think so, since both of my Win installations have English and Turkish keyboards installed, but it is not an area I have much expertise on. If there are some lines that can help debug the error, I can check it out as well.

arha commented 3 months ago

+1 maybe an issue with i18n. I also have a keep notes with multiple alphabets, tried an export on a win10, same issue. Works first try on a linux, everything was exported right.

djsudduth commented 3 months ago

@arha - do you know if the versions of Python were the same on Win10 and Linux?

djsudduth commented 3 months ago

I tried some turkish notes and was not able to get it to fail. I'm wondering about the missing titles possibly having something odd within them @ndilekli. Were the titles completely empty in the Win10 case where it failed?

ndilekli commented 3 months ago

I

I tried some turkish notes and was not able to get it to fail. I'm wondering about the missing titles possibly having something odd within them @ndilekli. Were the titles completely empty in the Win10 case where it failed?

I think so. Because I added titles to the top notes and the script processed more files.

djsudduth commented 2 months ago

@ndilekli - I'm going to close this one for now. If it pops up again - maybe you can figure out what caused it and we'll reopen the issue.