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

Script to convert CSV into HCC #81

Closed
kenorb opened this issue Oct 12, 2016 · 1 comment · Fixed by #82
Closed

Script to convert CSV into HCC #81

kenorb opened this issue Oct 12, 2016 · 1 comment · Fixed by #82
Assignees

Comments

@kenorb
Copy link
Member

kenorb commented Oct 12, 2016

Expand convert_csv_to_mt.py script to convert CSV input files into HCC (as implemented as part of GH-77). So -f parameter should accept the new hcc format (similar as it works for other hst/fxt formats).

codemill$120

@paulohrpinheiro
Copy link
Contributor

Hello, Rafal.

The script is already generating the file, I am now finishing the
writing of the headers (with the offset of the data and the number of
lines). I also started writing some tests with unittest. For this I had
to move some parts of the script to functions, so must also check that
did not cause side effects in the other options. Checking all this, I
send the PR to you test too. I hope to finish in the maximum tomorrow
morning (my time now is Mon, 17 Oct 2016 14:18:34 -0200).

Hug, thank you.

Paulo Henrique Rodrigues Pinheiro
https://about.me/paulohrpinheiro

Em 17-10-2016 14:11, Rafal escreveu:

@paulohrpinheiro https://github.com/paulohrpinheiro Let me know if
you've any questions and you're happy to work on the issue.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#81 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0MP3yFuIFCGZBnCAaETITY0qClNyq9ks5q054-gaJpZM4KU3-8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants