Skip to content
This repository was archived by the owner on Mar 16, 2023. It is now read-only.

Conversation

@LinearAlpha
Copy link

Fixed(?) how smi style color code converted to ass style.
Changed and test to run on python 3.8.x on linux_x86-64 and arm Also, changed to not add "kor" on the end of the file when there is no other language is depicted.
Also, add to print what file is currently working on.
Also, some setting updates.

LinearAlpha and others added 3 commits May 22, 2021 04:18
Fixed(?) how smi style color code converted to ass style.
Changed and test to run on python 3.8.x on linux_x86-64 and arm Also, changed to not add "kor" on the end of the file when there is no other language is depicted.
Also, add to print what file is currently working on.
Also, some setting update.
On the original code, the cover conversion  from smi to ass was wrong.
It add new function that converting RGB to BGR
Copy link
Owner

@trustin trustin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for your contribution!

Could you also revert the file mode changes in the .sh files?

/build
/test_smis/*.ass
__pycache__
.vscode No newline at end of file
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you revert this?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is automatically added by my IDE setting.

else:
print ('version 3.x needed')

default_lang_code = 'kor'
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you let me know why this had to be removed?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought if the subtitle file only incused single language in the subtitle it is meaningless to add kor on the file name. Since already know what language is in the subtitle file.


default_lang_code = 'kor'
default_font_name = 'sans-serif'
default_font_name = 'Malgun Gothic'
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about using fonts like "Noto Sans CJK KR" instead, given Malgun Gothic is not available in non-Windows?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is part of my mistake, while using and testing this program, I modified the ass to match with my subtitle settings.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants