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

test: relax allocated tests #468

Merged
merged 1 commit into from
Nov 8, 2023
Merged

test: relax allocated tests #468

merged 1 commit into from
Nov 8, 2023

Conversation

vtjnash
Copy link
Contributor

@vtjnash vtjnash commented Nov 8, 2023

Several of these are failing being slightly over the threshold on some CI runs. Relax all the thresholds, since the intent is to make sure allocations are completely dynamic, not to prohibit them entirely.

C.f. https://buildkite.com/julialang/julia-master/builds/29821#018bab25-9f95-4abd-aabf-6eaa8c05859e for example

Several of these are failing being slightly over the threshold on some
CI runs. Relax all the thresholds, since the intent is to make sure
allocations are completely dynamic, not to prohibit them entirely.
@ViralBShah ViralBShah merged commit 37fc321 into main Nov 8, 2023
6 checks passed
@ViralBShah ViralBShah deleted the jn/test-allocated-relax branch November 8, 2023 16:06
@ViralBShah
Copy link
Member

Should we bump SparseArrays on master to pull this in?

@vtjnash
Copy link
Contributor Author

vtjnash commented Nov 8, 2023

eventually yes

@vtjnash
Copy link
Contributor Author

vtjnash commented Nov 8, 2023

@ViralBShah
Copy link
Member

Yeah, my question was whether this was urgent and if we should do it right away.

@vtjnash
Copy link
Contributor Author

vtjnash commented Nov 8, 2023

it is causing a few CI to fail, but not the most frequent issue right away

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