search

matlab-deep-learning / transformer-models

Deep Learning Transformer models in MATLAB
Other
206 stars 61 forks source link

Updating predictMaskedToken.m to match new tokenizer. #33

Closed debymf closed 1 year ago

debymf commented 1 year ago

Updating predictMaskedToken.m to match new tokenizer. Also included test file to make sure we flag this in future submissions. The README was also changed to include pointers to where find the data sets used in the GitHub examples.

bwdGitHub commented 1 year ago

Feel free to merge after checking the one comment I made - I thought there was an "approve" button for GitHub PRs.

If you want to be pedantic you could create a bug Issue and close it with the PR. My thought there is maybe some users would like to see what bugs existed, and what commits/merges fix them.