Add demo: fit a line #130
Add demo: fit a line #130
Conversation
| @@ -0,0 +1,10 @@ | |||
| Metadata-Version: 1.0 | |||
There was a problem hiding this comment.
These files are not needed?
- PKG-INFO
- SOURCES.txt
- dependency_links.txt
- top_level.txt
|
|
||
| __all__=["train", "test", "fetch"] | ||
|
|
||
| CURRENT_DATACENTER = os.getenv("CURRENT_DATACENTER") |
There was a problem hiding this comment.
Environment variables should be named like "[APPLICATION]_[VARIABLE]" to avoid name conflicts. Like: PADDLE_CLOUD_CURRENT_DATACENTER
|
|
||
| CURRENT_DATACENTER = os.getenv("CURRENT_DATACENTER") | ||
|
|
||
| common.DATA_HOME = "/pfs/%s/public/dataset" % CURRENT_DATACENTER |
There was a problem hiding this comment.
Data dir should look like /pfs/%s/public/uci_housing?
There was a problem hiding this comment.
public is the public folder mount in Pod with readOnly access, and add a sub-folder dataset to storage all the datasets is help for adding others in the public folder. And also in PaddlePaddle dataset, the directory is ~/.cache/paddle/dataset/...
| def train(): | ||
| return common.cluster_files_reader( | ||
| TRAIN_FILES_PATTERN, | ||
| trainer_count = int(os.getenv("TRAINERS", "1")), |
There was a problem hiding this comment.
Read trainer_count from PADDLE_INIT_NUM_GRADIENT_SERVERS
docker/python/setup.py
Outdated
| 'paddle.cloud', | ||
| 'paddle.cloud.dataset'] | ||
|
|
||
| setup(name='pcloud', |
There was a problem hiding this comment.
Package name and import name should be the same.
There was a problem hiding this comment.
Done.
Change the package name paddle.cloud=> pcloud
demo/fit_a_line/train.py
Outdated
| @@ -0,0 +1,49 @@ | |||
| import paddle.v2 as paddle | |||
| import paddle.cloud.dataset.uci_housing as uci_housing | |||
There was a problem hiding this comment.
Do not use the name paddle, may conflict with original paddle project
Fixed #110