Skip to content

Quickfix requirements #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

tkukurin
Copy link

@tkukurin tkukurin commented Dec 15, 2024

Tried to run repo locally, got stopped at the requirements install step.

After changes in this PR, this works:

uv venv -p 3.9 .venv
uv pip install -p 3.9 -r requirements.txt
source .venv/bin/activate.fish
python run.py cogs/trafo.yaml

The PR fixes errors (root cause is always some place where Tensorboard is being annoying):

`np.object` was a deprecated alias for the builtin `object`. To avoid this error in existing code, use `object` by itself. Doing this will not modify any behavior and is safe.


from torch.utils.tensorboard.writer import SummaryWriter
AttributeError: module 'distutils' has no attribute 'version'


1. Downgrade the protobuf package to 3.20.x or lower.
TypeError: Descriptors cannot be created directly.

Alternative might be changing the pinned Tensorboard version but then probably other requirements become out-of-sync with original runs.

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