-
Notifications
You must be signed in to change notification settings - Fork 629
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
Adds backend / SDK dev doc #262
Adds backend / SDK dev doc #262
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very clear and straightforward and to-the-point documentation. Great stuff!
Any code changes in ```server``` folder, simply run | ||
|
||
``` | ||
python visualdl/server/visualDL --logdir={LOG_DIR} --port=8080 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So I had a problem on this step wherein it was importing libraries from the pip installed version of visualDL
. So I had to resolve it by resetting my PYTHONPATH. You don't need to explain it as much as it would be nice if it is mentioned in a comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ya, I run into same issue, let me update
docs/how_to_dev_backend_en.md
Outdated
|
||
#### Information Container | ||
- Currently it just handles logic of sync cycle (when to write) | ||
- In future we may add logic of when to read |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When to read needn't be synced or controlled, because the R/W conflicts happen rarely, and the web service can be retriggered a few times, that is a cheap but relatively not bad way to assure read service reliability.
The read service should not affect the writing service, that is the key.
docs/how_to_dev_backend_en.md
Outdated
- ```Record``` : represent any data structure for any component | ||
- Handles the actual logic to serialized to disk and deserialized from disk | ||
- ```storage.proto```: protobuf file that defines the exact interface to be read/write | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove the empty line in the tail.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great stuff. Thanks
No description provided.