search

stickeritis / sticker

Succeeded by SyntaxDot: https://github.com/tensordot/syntaxdot
Other
25 stars 2 forks source link

Batch size in config #151

Closed twuebi closed 4 years ago

twuebi commented 4 years ago

Changing the batch size on the fly is a nice convenience when experimenting. Does something speak against moving it out of the config file?

danieldk commented 4 years ago

Actually, it is already an option for pipeline commands (tag and server), the configuration file is ignored. I would be completely in favor of doing the same for training commands.

If we remove it though, we should make it optional in the config and give a warning when it is still in the configuration (like read_ahead now).

danieldk commented 4 years ago

I'll assign myself for this ;).