nest-simulator icon indicating copy to clipboard operation
nest-simulator copied to clipboard

Fix issue 3446

Open JanVogelsang opened this issue 1 year ago • 1 comments

This PR adds a regression test for issue #3446. Please fork this branch to implement a fix for the issue.

JanVogelsang avatar Mar 27 '25 16:03 JanVogelsang

Pull request automatically marked stale!

github-actions[bot] avatar May 27 '25 08:05 github-actions[bot]

Is this still active? If so: Could you comment on why a fix should not be based on latest master? What other changes are injected when starting from your branch, as you suggest?

terhorstd avatar Jul 11 '25 13:07 terhorstd

This branch, which is based on master, makes the CI fail by adding a regression test.

JanVogelsang avatar Jul 17 '25 18:07 JanVogelsang

We cannot merge changes that cause the CI to fail. Is someone looking into implementing a fix for the underlying issue?

terhorstd avatar Aug 13 '25 15:08 terhorstd

We cannot merge changes that cause the CI to fail. Is someone looking into implementing a fix for the underlying issue?

This raises actually an interesting question: How can we collect reproducers/regression tests for issues that we have on the back-burner? Maybe for now it would make most sense to change the title of this PR to "Add regression test for issue 3446" and convert it to Draft. Then, when someone starts to work on an actual solution, they could pull this PR/branch into their own work. In this way, we would at least make it openly visible that the regression test is already available.

heplesser avatar Aug 14 '25 06:08 heplesser

This PR will stay available and is linked directly from the issue. There is no use having it as draft and continuously occupy our attention on the PR board. It's still an issue, hence we find it in the issues. I'll close this for now, please re-open when works are picked up again.

terhorstd avatar Aug 27 '25 08:08 terhorstd