Skip to content

Dockerfile fix and improvements#209

Open
lorenzosciuto wants to merge 1 commit into
nvidia-cosmos:mainfrom
lorenzosciuto:main
Open

Dockerfile fix and improvements#209
lorenzosciuto wants to merge 1 commit into
nvidia-cosmos:mainfrom
lorenzosciuto:main

Conversation

@lorenzosciuto

Copy link
Copy Markdown

1.Installed nasm: Added nasm to the apt-get install list. This provides the necessary assembler for tensorstore to compile its low-level code, preventing the segmentation fault.

2.Consolidated RUN Layers: Merged several RUN commands for system package installation and symlinking into a single, more efficient Docker layer.

3.Removed Redundancy: Original script called pip install decord==0.6.0 twice, removed the second instance.

1.Installed nasm: Added nasm to the apt-get install list. This provides the necessary assembler for tensorstore to compile its low-level code, preventing the segmentation fault.

2.Consolidated RUN Layers: Merged several RUN commands for system package installation and symlinking into a single, more efficient Docker layer.

3.Removed Redundancy: Original script called pip install decord==0.6.0 twice, removed the second instance.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant