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

temporary fix to error: array of sample points empty #1111

Closed
wants to merge 9 commits into from

Conversation

AdiBish
Copy link

@AdiBish AdiBish commented Jul 7, 2024

No description provided.

@marius10p
Copy link
Collaborator

What is the error here? @iamzoltan can we please prioritize merging this and rerunning the book? I have also made some direct commits to the main branch with links that need to get updated in the jupyterbook (for both CN and DL).

@iamzoltan
Copy link
Contributor

@marius10p you should not be making direct commits to the main branch. Doing so skips part of our processing pipeline and depending on what you changed, they may not be included in the next build.

These checks failed because forked repos should not be making PRs into main, but into a staging branch instead. This way we can create a PR to main from that staging branch to allow the processing pipeline to run.

@iamzoltan
Copy link
Contributor

Closing this PR as it is address in the allen-hot-fix pr

@iamzoltan iamzoltan closed this Jul 8, 2024
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