Skip to content

Conversation

@booxter
Copy link
Contributor

@booxter booxter commented Apr 15, 2025

This is to work around an issue in flash-attn build definition. The
issue manifests itself as:

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [6 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-install-u40hct41/flash-attn_36e9333b3570426a854895a477ce846d/setup.py", line 22, in <module>
          import torch
      ModuleNotFoundError: No module named 'torch'
      [end of output]

See: Dao-AILab/flash-attention#958

Signed-off-by: Ihar Hrachyshka [email protected]

@mergify mergify bot added CI/CD Affects CI/CD configuration ci-failure labels Apr 15, 2025
@booxter
Copy link
Contributor Author

booxter commented Apr 15, 2025

FYI because the workflow uses pull_request_target, the change doesn't apply in CI, hence the failure.

This is to work around an issue in flash-attn build definition. The
issue manifests itself as:

```
  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [6 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-install-u40hct41/flash-attn_36e9333b3570426a854895a477ce846d/setup.py", line 22, in <module>
          import torch
      ModuleNotFoundError: No module named 'torch'
      [end of output]
```

See: Dao-AILab/flash-attention#958

Signed-off-by: Ihar Hrachyshka <[email protected]>
@mergify mergify bot added the one-approval label Apr 16, 2025
@booxter booxter requested a review from JamesKunstle April 16, 2025 20:39
@JamesKunstle JamesKunstle merged commit e65d21c into instructlab:main Apr 16, 2025
7 of 9 checks passed
@mergify mergify bot removed the one-approval label Apr 16, 2025
@booxter
Copy link
Contributor Author

booxter commented Apr 19, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Affects CI/CD configuration ci-failure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants