just a small problem in setup.md

Open
  • Is something wrong?
#173 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
58/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Stale
Domain
documentation

Research direction

Open setup.md and locate the prerequisites section containing the VRAM requirement. Correct the unit notation from "32Gb" to "32GB", then review the rendered Markdown and diff to confirm that only the documented prerequisite changed.

Written by the indexing model from the issue text.

Description

The prerequisites in setup.md says "32Gb vRAM", but it should be "32GB vRAM" instead. Because 32Gb equals to 4GB, the difference in letters may mislead researchers who want to use the model to deploy it on a GPU with insufficient VRAM, which can cause "CUDA out of memory" error.

Dominant language
Python
Stars
7.6k
Forks
898
No merged PRs in 30d

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from facebookresearch/sam-3d-objects

All issues in facebookresearch/sam-3d-objects

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.