fix: small qol improvements to devcontainer (#1427)

Signed-off-by: Alec <35311602+alec-flowers@users.noreply.github.com>
Co-authored-by: Ryan McCormick <rmccormick@nvidia.com>

.devcontainer/README.md

@@ -17,7 +17,7 @@ limitations under the License.
 
 # NVIDIA Dynamo Development Environment
 
-> Warning: devcontainers is an experimental feature and we are not testing in CI. Please submit any feedback using the issues on github.
+> Warning: devcontainers is an experimental feature and we are not testing in CI. Please submit any feedback using the issues on GitHub.
 
 ## Prerequisites
 - [Docker](https://docs.docker.com/get-started/get-docker/) installed and configured on your host system
@@ -51,7 +51,23 @@ Make sure HF_HOME is sourced in your .bashrc or .zshenv and your vscode default
 - The container will mount your local code
 - `post-create.sh` will build the project and configure the environment
 
-If `post-create.sh` fails, you can try to debug or [submit](https://github.com/ai-dynamo/dynamo/issues) an issue on github.
+If `post-create.sh` fails, you can try to debug or [submit](https://github.com/ai-dynamo/dynamo/issues) an issue on GitHub.
+
+## Development Flow
+
+If you make changes to Rust code and want to compile, use [cargo build](https://doc.rust-lang.org/cargo/commands/cargo-build.html). This will update Rust binaries such as dynamo-run.
+
+```bash
+cd /home/ubuntu/dynamo && cargo build --locked --profile dev
+```
+
+Before pushing code to GitHub, remember to run `cargo fmt` and `cargo clippy`
+
+If you make changes to Rust code and want to propagate to Python bindings then can use [maturin](https://www.maturin.rs/#usage) (pre-installed). This will update the Python bindings with your new Rust changes.
+
+```bash
+cd /home/ubuntu/dynamo/lib/bindings/python && maturin develop
+```
 
 ## What's Inside
 Development Environment:
@@ -80,6 +96,15 @@ Edit `.devcontainer/devcontainer.json` to modify:
 - Container configuration
 - Custom Mounts
 
+## Documentation
+
+To look at the docs run:
+```bash
+cd ~/dynamo/.build/target/doc && python3 -m http.server 8000
+```
+
+VSCode will automatically port-forward and you can check them out in your browser.
+
 ## FAQ
 
 ### GPG Keys for Signing Git Commits

.devcontainer/devcontainer.json

@@ -32,7 +32,6 @@
         "vscode": {
             "extensions": [
                 "ms-python.python",
-                "ms-python.pylint",
                 "ms-python.vscode-pylance",
                 "rust-lang.rust-analyzer"
             ],
@@ -42,10 +41,24 @@
 
                 "python.defaultInterpreterPath": "/opt/dynamo/venv/bin/python",
                 "python.linting.enabled": true,
-                "python.linting.pylintEnabled": true,
 
+                "rust-analyzer.memoryLimit": 4096, // larger memory limit to reduce latency
                 "rust-analyzer.checkOnSave.command": "clippy",
                 "rust-analyzer.checkOnSave.enable": true,
+                "rust-analyzer.cargo.buildScripts.enable": true,
+                "rust-analyzer.cargo.targetDir": "/home/ubuntu/dynamo/.build/target",
+                "rust-analyzer.procMacro.enable": true,
+                "rust-analyzer.completion.autoimport.enable": true,
+
+                // Enhanced rust-analyzer configuration
+                "rust-analyzer.linkedProjects": [
+                    "dynamo/Cargo.toml",
+                    "dynamo/lib/runtime/Cargo.toml",
+                    "dynamo/lib/llm/Cargo.toml",
+                    "dynamo/lib/tokens/Cargo.toml",
+                    "dynamo/lib/bindings/python/Cargo.toml",
+                    "dynamo/launch/dynamo-run/Cargo.toml"
+                ],
 
                 "files.trimTrailingWhitespace": true,
                 "files.insertFinalNewline": true

.devcontainer/post-create.sh

@@ -58,17 +58,15 @@ ln -sf $HOME/dynamo/.build/target/debug/dynamo-run $HOME/dynamo/deploy/sdk/src/d
 ln -sf $HOME/dynamo/.build/target/debug/http $HOME/dynamo/deploy/sdk/src/dynamo/sdk/cli/bin/http
 ln -sf $HOME/dynamo/.build/target/debug/llmctl $HOME/dynamo/deploy/sdk/src/dynamo/sdk/cli/bin/llmctl
 
-# install the python bindings in editable mode
-cd $HOME/dynamo/lib/bindings/python && retry uv pip install -e .
+# install the python bindings
+cd $HOME/dynamo/lib/bindings/python && retry maturin develop
+
+# installs overall python packages, grabs binaries from .build/target/debug
 cd $HOME/dynamo && retry env DYNAMO_BIN_PATH=$HOME/dynamo/.build/target/debug uv pip install -e .
 
 export PYTHONPATH=/home/ubuntu/dynamo/components/planner/src:$PYTHONPATH
 
-# Add to bashrc only if not already present
-if ! grep -q "source /opt/dynamo/venv/bin/activate" ~/.bashrc; then
-    echo "source /opt/dynamo/venv/bin/activate" >> ~/.bashrc
-fi
-
+# TODO: Deprecated except vLLM v0
 if ! grep -q "export VLLM_KV_CAPI_PATH=" ~/.bashrc; then
     echo "export VLLM_KV_CAPI_PATH=$HOME/dynamo/.build/target/debug/libdynamo_llm_capi.so" >> ~/.bashrc
 fi

container/Dockerfile.vllm

@@ -308,7 +308,7 @@ RUN chown $USERNAME:$USERNAME /opt/dynamo/venv
 COPY --from=base --chown=$USERNAME:$USERNAME /usr/local/bin /usr/local/bin
 
 # so we can use maturin develop
-RUN uv pip install maturin
+RUN uv pip install maturin[patchelf]
 
 USER $USERNAME
 ENV HOME=/home/$USERNAME