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.

Sign up for GitHub

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

Jump to bottom

Test allocations #139

Open

albert-de-montserrat wants to merge 4 commits into main from adm-test-allocations

Member

albert-de-montserrat commented Dec 4, 2023

After the type stability improvements, we can uncomment the old @test num_alloc <= 32 and replace them by @test num_alloc == 0

albert-de-montserrat added 4 commits

December 5, 2023 00:09


          test allocations

10b2602


          use AllocCheck.jl

b9b5ba6


          fix tests not working on Windows

693e63e


          fix broken test in Windows

05a2537

Member Author

albert-de-montserrat commented Dec 5, 2023

I will put this on hold, AllocChecks.jl doesn't seem to be very robust in Windows yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet