Skip to content

Conversation

@dnwpark
Copy link
Contributor

@dnwpark dnwpark commented Aug 8, 2025

No description provided.

@dnwpark dnwpark requested a review from 1st1 August 8, 2025 17:44
@dnwpark
Copy link
Contributor Author

dnwpark commented Aug 8, 2025

My setup issue was because my environment was messed up.

```bash
pip install setuptools==70
pip install cython
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wait... pip install -e. should take care of these, no?

cython is in pypackage.toml

[build-system]
requires = ["Cython(>=3.1.1,<4.0.0)", "setuptools >= 67", "wheel"]
build-backend = "setuptools.build_meta"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, it's weird, I had to manually install these

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe my env is still weird 🫠

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let me try some stuff

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, I always have to do that too.

@1st1
Copy link
Member

1st1 commented Aug 8, 2025

Add a note about your warnings -- add a section "gotchas" or FAQ, copy/paste your warnings, add the recommendation + explanation (pyx files were changed, run make clean && make to force-recompile all cython code)

@dnwpark dnwpark changed the title Add note about extra build requirements in arch.md. Add a setup gotchas section to arch.md. Aug 8, 2025
@dnwpark
Copy link
Contributor Author

dnwpark commented Aug 8, 2025

@1st1 I couldn't reproduce some of the earlier errors, but hopefully the next person to encounter them will take better notes.

@1st1
Copy link
Member

1st1 commented Aug 8, 2025

@1st1 I couldn't reproduce some of the earlier errors

just copy paste from slack, errors are always the same and the cause is well-known

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.

4 participants