Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for UTF-16 character encoding #17

Open
denniskhk opened this issue Jun 1, 2021 · 6 comments
Open

Add support for UTF-16 character encoding #17

denniskhk opened this issue Jun 1, 2021 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@denniskhk
Copy link

This utility is extremely useful - I used to have to manually parse and enter the info into a spreadsheet by hand :((

This works great for logs created by the Chia UI logger. However, I have recently started to plot using the CLI. I pipe the output into a log via Powershell tee command. This produces a similar file BUT one major difference is that it does not have the entire block at the top - it starts with the "Starting plotting progress into temporary dirs: xxx" line onwards. I've attached a sample log file.
_202105312205.log

This causes the utility to just crash out with no output.
Perhaps it would be great if the utility could be modified so it'll skip missing info (treat as blank) and move on to parse the remaining info?

Thanks again for this useful time-saving tool!

@denniskhk denniskhk added the enhancement New feature or request label Jun 1, 2021
@Pengor Pengor self-assigned this Jun 1, 2021
@Pengor Pengor added bug Something isn't working and removed enhancement New feature or request labels Jun 1, 2021
@Pengor
Copy link
Owner

Pengor commented Jun 1, 2021

Hi there, unfortunately I was not able to reproduce your issue, on my PC I was able to parse the sample file you attached without any problems. The first line that Chia Log Analysis looks for is Starting plotting progress into temporary dirs: dir_1 dir_2 so I would expect the format to work just fine. Could you share these additional details and I can help troubleshoot why it's not working for you?

  • Which operating system are you using?
  • How did you format the command?
  • What is the filepath that the log file resides in?

@Pengor Pengor added the question Further information is requested label Jun 1, 2021
@denniskhk
Copy link
Author

denniskhk commented Jun 2, 2021 via email

@Pengor Pengor removed the question Further information is requested label Jun 7, 2021
@Pengor Pengor changed the title Feature request: skip missing info Issue parsing CLI plotting logs Jun 7, 2021
@Pengor
Copy link
Owner

Pengor commented Jun 7, 2021

Okay it would seem the issue is that the PowerShell plotter log file does not use the more common UTF-8 character encoding (which the other plotter logs do), I will look into adding support for 16-bit character encodings.

@Pengor Pengor changed the title Issue parsing CLI plotting logs Add support for UTF-16 character encoding Jun 7, 2021
@Pengor Pengor added enhancement New feature or request and removed bug Something isn't working labels Jun 7, 2021
@denniskhk
Copy link
Author

denniskhk commented Jun 7, 2021 via email

@Pengor
Copy link
Owner

Pengor commented Jun 7, 2021

Yes, I was able to parse the log once it was converted to UTF-8. I think this is why I was unable to reproduce your issue originally, I probably copied and pasted the log file's contents (which likely converted it to UTF-8) rather than use the file itself.

I also noticed the line endings in your log file were CRLF (Windows style) whereas the GUI plotter produces LF line endings (Unix style) but the application was able to handle both just fine so that is a non-issue.

@denniskhk
Copy link
Author

denniskhk commented Jun 8, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants