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

Kikongo MT #124

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

Kikongo MT #124

wants to merge 13 commits into from

Conversation

berthine
Copy link

Kikongo is one of the language spoken in DRC.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

Review Jupyter notebook visual diffs & provide feedback on notebooks.


Powered by ReviewNB

Copy link
Collaborator

@juliakreutzer juliakreutzer left a comment

Choose a reason for hiding this comment

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

Awesome, great job! 💯

To clean up the files, could you please remove the .ipynb_checkpoints directory from the PR? And the starter_notebook.ipynb, otherwise it will overwrite the default one.

In addition, please upload the best.ckpt for the model on GoogleDrive (or a similar platform), and add the link to the README. This will ensure that others can load and use the model as well. Thank you! ❤️

@berthine
Copy link
Author

berthine commented Aug 28, 2020 via email

@berthine
Copy link
Author

berthine commented Sep 8, 2020

Done.

Copy link
Collaborator

@juliakreutzer juliakreutzer left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you!

One last thing: please remove the "starter_notebook" from the PR, otherwise it gets overwritten.

@berthine
Copy link
Author

berthine commented Sep 28, 2020 via email

@berthine
Copy link
Author

berthine commented Oct 3, 2020

Done.

Copy link
Collaborator

@juliakreutzer juliakreutzer left a comment

Choose a reason for hiding this comment

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

Hi @berthine sorry for the misunderstanding, but now your commit would remove the starter_notebooks from the master branch as well. Can you please revert those files? See https://www.git-tower.com/learn/git/faq/restoring-deleted-files/

@juliakreutzer
Copy link
Collaborator

Same for the starter_notebook-custom-data.ipynb.

@juliakreutzer
Copy link
Collaborator

@berthine ping :)

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.

2 participants