Bump langevin solver to 1.4.7#1711
Merged
Merged
Conversation
Picks up the fix for the NullPointerException when a bound site transitions to a state with no defined binding reaction (cam-center/LangevinNoVis01#36, release 1.4.7). vcell-core downloads the per-platform binaries from the GitHub release using these version properties, and Dockerfile-batch-dev COPYs the downloaded linux binary into the vcell-batch image. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates the Langevin (SpringSaLaD) solver from 1.4.6 → 1.4.7.
What 1.4.7 fixes
Release 1.4.7 fixes a
NullPointerExceptionthat crashed the solver when a bound site transitioned to a(state, partner-state)signature with no associated binding reaction (cam-center/LangevinNoVis01#36). The bond now retains its existing reaction identity/length and stops spontaneously dissociating in that state (off-rate0.0) instead of crashing.Change
Bumps the three solver version properties in
pom.xml:solvers-langevin-mac.versionsolvers-langevin-windows.versionsolvers-langevin-linux.versionHow this propagates
vcell-core/pom.xmldownloads the per-platform binaries from the GitHub release using these properties (langevin-windows-latest,langevin-macos-universal,langevin-ubuntu-latest) intolocalsolvers/.docker/build/Dockerfile-batch-devCOPYs the downloadedlocalsolvers/linux64/langevin_x64into the vcell-batch image (and onward to the singularity image) — no Dockerfile change required.Per request,
vcell-fluxcdis intentionally not changed in this PR.🤖 Generated with Claude Code