Skip to content

Commit c60dc6d

Browse files
authored
1167 fixes protobuf (#1171)
Fixes #1167 ### Description - pip install ray might upgrade protobuf automatically, but protobuf v4 is not compatible with some versions of pytorch's tensorboard summarywriter ### Checks <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [ ] Avoid including large-size files in the PR. - [ ] Clean up long text outputs from code cells in the notebook. - [ ] For security purposes, please check the contents and remove any sensitive info such as user names and private key. - [ ] Ensure (1) hyperlinks and markdown anchors are working (2) use relative paths for tutorial repo files (3) put figure and graphs in the `./figure` folder - [ ] Notebook runs automatically `./runner.sh -t <path to .ipynb file>` Signed-off-by: Wenqi Li <[email protected]>
1 parent 5a3f5ee commit c60dc6d

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

deployment/ray/mednist_classifier_ray.ipynb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
},
4242
"outputs": [],
4343
"source": [
44-
"%pip install ray[serve]\n",
44+
"!pip install \"ray[serve]\" \"protobuf<4\"\n",
4545
"!python -c \"import monai\" || pip install -q \"monai-weekly[gdown, tqdm]\""
4646
]
4747
},

runner.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,7 @@ for file in "${files[@]}"; do
517517
then
518518
export PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python
519519
else
520-
export PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=
520+
unset PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION
521521
fi
522522

523523
cmd=$(echo "papermill ${papermill_opt} --progress-bar -k ${kernelspec}")

0 commit comments

Comments
 (0)