Skip to content

Commit be49848

Browse files
authored
Add missing " in S3 and add prerequisite for tracing in S4 (#492)
* Add missing " in Docker markdown * Add uv.lock to Docker image essentials * Add prerequisite for tracing
1 parent a3cd236 commit be49848

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

s3_reproducibility/docker.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@ beneficial for you to download.
260260
3. Finally, we are going to name our training script as the *entrypoint* for our Docker image. The *entrypoint* is
261261
the application that we want to run when the image is executed:
262262
263-
=== "Using pip
263+
=== "Using pip"
264264
265265
```docker
266266
ENTRYPOINT ["python", "-u", "src/<project-name>/train.py"]

s4_debugging_and_logging/profiling.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,7 @@ result nicely (like `snakeviz` for `cProfile`) we are also going to use the tens
247247
prof.export_chrome_trace("trace.json")
248248
```
249249

250-
You should be able to visualize the file by going to `chrome://tracing` in any chromium-based web browser.
250+
You should be able to visualize the file by going to `chrome://tracing` in any chromium-based web browser (provided you have enabled internal debugging pages at chrome://chrome-urls).
251251
Can you still identify the information printed in the previous exercises from the visualizations?
252252

253253
5. Running profiling on a single forward step can produce misleading results, as it only provides a single sample that

0 commit comments

Comments
 (0)