facebookresearch / facebookresearch/perception_models

None pointer in rope.py

Open
#119 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Jupyter Notebook
Stars
2.4k
Forks
162
PR merge metrics
No merged PRs in 30d

Description

Hello,

I've encountered some strange behavior in rope.py when using the perception models with Torch 2.9.1+cu130, CUDA 13.0, and Python 3.11.14 in a container.

I made a small change to line 340 to check that the "freq" attribute is correctly defined:

if self.freq is not None: self.freq = self.freq.to(device)

I don't know if this is optimal, but it works for me.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.