unicode parsing issues for sample data provided?

[fractaloutlook]

Heyo! Not sure if this project is still active, but:

When running it, 1) - the output messaging seems to claim that it is ignoring the ledgers I asked for... and 2) the execution end with an error about not being able to decode a character to unicode format?

It has that same error with or without the ledgers parameter, or using any other parameters. I downloaded the sample data for bitcoin, tezos, cardano, and ethereum from the page linked in the readme.

Any thoughts on this one?

[LadyChristina]

Hello @fractaloutlook, thanks for raising this to our attention.

For your first point, we very recently changed the way parameters are specified and I only now realise that we forgot to update the documentation, sorry about that! We will update the docs asap, but basically all the configurations are now done through the config.yaml file. So if you want only one ledger to be included, you would need to comment out the rest. Let me know if you have any questions on this.

For your second issue, I'm not sure why this is happening, it's not something I can reproduce right now. Can you confirm that you haven't made any changes to this file on your local copy?

[fractaloutlook]

Yep, my file is the same, I never made any changes to it. I've got my yaml file pared down to just the 4 I have data for.... and it still failed at the same point. I then commented out bitcoin from the list, and Got this different error:

I was running cardano, ethereum, tezos. ran the py file with no parameters.

[LadyChristina]

Thanks for the response. I think I figured out the issue - we have to explicitly specify the encoding when reading the identifier files in order for them to be parsed on Windows. I have opened a PR that solves this issue, I will let you know once it is approved and merged into the main branch

[fractaloutlook]

Great thanks! Would the instructions and files as-they-are have worked in WSL Ubuntu? Just curious.

[LadyChristina]

Yes, they should work in WSL as they are

[LadyChristina]

Fyi the PR has now been merged so if you pull you should be able to run the code on Windows too (but do let us know if any problems persist!)