Skip to content

Fix sampling tests (and others)#1037

Merged
Zapiano merged 5 commits intomainfrom
fix/sampling-tests
Feb 6, 2026
Merged

Fix sampling tests (and others)#1037
Zapiano merged 5 commits intomainfrom
fix/sampling-tests

Conversation

@Zapiano
Copy link
Member

@Zapiano Zapiano commented Feb 6, 2026

  • Fix bug caused by mismatch between color types (RGBA{Float32} and RGBA{Float64})
  • Adds calls to GC in test/data_loading.jl. An error was occurring there, likely caused by unclosed file handles between calls.
  • Fix test/sampling.jl after introduction of reactive intervention strategy parameters. The sampling method sometimes changes the values of these parameters to zero, even when though some of them have lower sampling bound greater than zero. In those cases, these parameters are not being tested.

This is to prevent the many problems because colors can be either Strings, Symbols or RGBA, depending on the context.
Some tests that check if the factors are withing sampling bounds stopped working after reactive intervention strategies were introduced. That's because the sampling method sometimes changes their values to zero even when the lower sampling bound is greater than zero. In those cases, I am not  testing these parameters.
@Zapiano Zapiano self-assigned this Feb 6, 2026
@Zapiano Zapiano added the bug Something isn't working label Feb 6, 2026
@Zapiano Zapiano changed the title Fix tests Fix sampling tests (and others) Feb 6, 2026
@Zapiano Zapiano merged commit 81a08a5 into main Feb 6, 2026
2 checks passed
@Zapiano Zapiano deleted the fix/sampling-tests branch February 6, 2026 06:39
@Zapiano
Copy link
Member Author

Zapiano commented Feb 6, 2026

@ConnectedSystems FYI

@ConnectedSystems
Copy link
Collaborator

ConnectedSystems commented Feb 6, 2026

I suggest adding a prefix to the "TEST" flag as these are environment variables and can be set by any program. Leads to confusion/conflict. Other env flags start with "ADRIA_"

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants