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.

Sign up for GitHub

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

Jump to bottom

Add library config interface #7

Open
rdadolf opened this issue Aug 1, 2016 · 0 comments
Open

Add library config interface #7

rdadolf opened this issue Aug 1, 2016 · 0 comments
Labels
L

Comments

@rdadolf
Copy link
Owner

rdadolf commented Aug 1, 2016

Fathom needs to be configured with external files for realistic runs. We need a configuration interface for this.

The right answer is probably to use ConfigParser and expect a config file in the current directory and/or the user's home directory.
@rdadolf rdadolf added this to the Release 1.0 milestone Aug 1, 2016
@rdadolf rdadolf mentioned this issue Aug 2, 2016
Closed
@rdadolf rdadolf modified the milestone: Release 1.0 Sep 15, 2016
@rdadolf rdadolf added the L label Oct 5, 2016
rdadolf added a commit that referenced this issue Oct 11, 2016
@rdadolf
Partial fixes for auto-download in seq2seq.
858bddf 
Seq2seq downlaods data to a '/data/WMT15' directory, but if a /data
directory didn't exist previously, then the call would fail with a less-
than-helpful error message.
This adds recursive directory creation, which also gives a better error
message. The hardcoded directory fix will have to wait for #7.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
L
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rdadolf