search

SpectralSequences / sseq

The root repository for the SpectralSequences project.
Apache License 2.0
22 stars 10 forks source link

Fix regression when `SECONDARY_JOB` environment variable is set #138

Closed JoeyBF closed 9 months ago

JoeyBF commented 9 months ago

It was introduced by me in #95 because the code preserved the t coordinate instead of the expected n

hoodmane commented 9 months ago

Could you add a test?

JoeyBF commented 9 months ago

Could you add a test?

Since we want to test the secondary example itself, I figured I would write a bash script that runs it and checks for the existence and nonexistence of the proper files. I put that in the Makefile, and I checked that that test fails without the current PR