Skip to content

[DO NOT MERGE] test CI#289

Open
ca16 wants to merge 11 commits into
mainfrom
chloea-test-ci
Open

[DO NOT MERGE] test CI#289
ca16 wants to merge 11 commits into
mainfrom
chloea-test-ci

Conversation

@ca16

@ca16 ca16 commented May 6, 2026

Copy link
Copy Markdown
Contributor

Trying to see if CI fails off the code in main. (and then some testing to see what might fix it)

Comment thread .github/workflows/mmda-ci.yml Outdated
- uses: actions/setup-python@v2
with:
python-version: '3.8'
python-version: '3.9'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This makes the test_word_predictors CI job pass. claude code says it was failing because 'pip can't find a pre-built wheel for your CI's Python 3.8'. And 3.9 is one of the versions we look at in the build CI job above so bumping here seems sort of okay?

Comment thread pyproject.toml Outdated
'smashed==0.1.10',
]
vila_predictors = [
'timm==0.9.12',

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This gets the test_vila_predictors CI job to fail with an error similar to the one we were hitting after the dropbox files disappeared.

Comment thread pyproject.toml Outdated
]
citation_links = [
'numpy',
'numpy==1.24.4',

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This gets us past errors in the build (3.9) job that look like this:

ERROR tests/test_utils/test_tools.py - ValueError: numpy.dtype size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject

claude code suggested it was a numpy version issue, and I picked this version because it's what's in the surviving task

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