-
Notifications
You must be signed in to change notification settings - Fork 4
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
src/config.json and src/test.py both zero-length files? #1
Comments
Hi @dweekly, You are right, they are empty. It is on my to-do list to refactor this code to create train/test scripts instead of jupyter notebooks. Best regards, |
Ok - it's just that the documentation said to use those files so my
expectation was that those had sample configurations and code to run. It's
ok if it's not done yet - I'd suggest just throwing in a comment that these
are a TODO and have an ETA of Jan so others don't end up similarly
confused. :)
Thank you!!
On Thu, Dec 3, 2020 at 16:25 Heitor Guimarães ***@***.***> wrote:
Hi @dweekly <https://github.com/dweekly>,
You are right, they are empty. It is on my to-do list to refactor this
code to create train/test scripts instead of jupyter notebooks.
The codes are presented in the nbs folder.
Unfortunately, I will not have time to implement it this year yet, but I
expect to do so in January/2021 (I'm working on a second version too, but I
don't have a fixed date for that yet).
Best regards,
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADMSYJ2QVTMIXQ446VEHZTSTBCBVANCNFSM4ULHK5XA>
.
--
Cheers,
David E. Weekly (@dweekly)
|
Yes, you are totally right. Sorry for this inconvenience, I will fix that. Cheers, |
It seems the GitHub repo may have had an issue at checkin because both the files in src/ have zero length.
The text was updated successfully, but these errors were encountered: