Skip to content
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

Update test.yml on new branch #14

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Update test.yml on new branch #14

wants to merge 5 commits into from

Conversation

kassabry
Copy link
Collaborator

@kassabry kassabry commented Jan 5, 2024

The old branch will be removed. This should maintain the history of the test.yml file that was lost with the other branch creation.

@agitter
Copy link
Member

agitter commented Jan 8, 2024

Do we know how to work around this error:

UnboundLocalError: local variable 'chunk' referenced before assignment

I believe we discussed that on one of our recent calls.

The Windows and macOS jobs are cancelled when the Ubuntu job hits an error. I believe we can try setting fail-fast to false in the matrix as described here.

kassabry and others added 4 commits January 10, 2024 09:17
Updated the BT_Clustering code line
Added the unzipping to the github actions which I believe was causing the problem with chunk referenced before assignment
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.

2 participants