-
Notifications
You must be signed in to change notification settings - Fork 106
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
Run script loads training dataset #67
Comments
Where is this happening? |
I am trying to run this now, maybe I've got it wrong. I've run the training from commit cc1ec7d. I use the same ini script that I used for training (see attachment It seems the dataset arguments are not ignored. I am also getting errors probably caused by the dataset init helper:
val_datasets.ini.txt NOTE the out_...out is just a thing I tried.. before this change there was the same error with s..._out keys.. |
It seems I've figured out the error. There was a leftover Python 2 matching of the regex. |
Is this solved? |
It shoudl be solved in the experiment object branch. (#73) (berliner refactor) |
I don't think it happens any more, though I don't when it got fixed. |
AFAIK it's not fixed.. I use two separate configuration files for this.
2016-11-30 13:19 GMT+00:00 Jindřich Libovický <notifications@github.com>:
… I don't think it happens any more, though I don't when it got fixed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#67 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABwcs1DK6m8Wuoej7sEZPl9gT75Bow2Lks5rDXfngaJpZM4JLO10>
.
|
According to the tests cases logs it does not happen any more. |
:D :D cože? :-) |
:-D už je to dobře |
škoda.. :-) |
Isn't it somehow connected with #210 ? |
Not at all. This was about handling configuration in |
Runner should not load the training dataset again.
The text was updated successfully, but these errors were encountered: