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

Nghiauet add comment for clarify #196

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Nghiauet
Copy link

Hi, I am reading the paper and trying to reimplement the code. But some functions seem to lack descriptions and hard to understand what going on.
What change:
Add some comments for perfect_align and is_sent_ok functions
What impact:
Don't change the logic code.

I still want to add more comments in the code. Please provide your instructions for me to make better explain for this code. Thanks

Nghiauet and others added 7 commits August 10, 2023 16:51
Fix typo in the code comments and update the gitignore file for vscode.

- Updated gitignore file to exclude the ".vscode/" directory.
- Fixed a typo in the comments of the code. The word "calculate" was misspelled as "caculate".
- Fixed another typo in the comments of the code. The word "return" was misspelled as "reutrn".
- Fixed a typo in the comments of the code. The word "tagged" was misspelled as "taged".
- Fixed a typo in the comments of the code. The word "delete" was misspelled as "deletes".
- Fixed a typo in the comments of the code. The word "separately" was misspelled as "separatly".
- Fixed a typo in the comments of the code. The word "return" was misspelled as "reutrn".
- Fixed a typo in the comments of the code. The word "alignment" was misspelled as "elignment".
- Fixed a typo in the comments of the code. The word "alignment" was misspelled as "alignment".
- Fixed a typo in the comments of the code. The word "INSERT" was misspelled as "INSERT".
- Fixed a typo in the comments of the code. The word "REPLACE_" was misspelled as "REPLACE".
- Fixed a typo in the comments of the code. The word "alignment" was misspelled as "alignmnet".
- Fixed a typo in the comments of the code. The word "list of source tokensq" was misspelled as "list of source tokens".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant