From 5d5747968b43e3d3391ba5376f4027eba2501605 Mon Sep 17 00:00:00 2001 From: "Daniel R. Roe" Date: Thu, 4 Jun 2026 09:31:36 -0400 Subject: [PATCH 1/2] Use Frame::SetFrame() instead of copying; previously only coordinate information was being properly copied. --- src/Action_SetVelocity.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/Action_SetVelocity.cpp b/src/Action_SetVelocity.cpp index 82cc6cee9c..fad13de412 100644 --- a/src/Action_SetVelocity.cpp +++ b/src/Action_SetVelocity.cpp @@ -140,10 +140,7 @@ Action::RetType Action_SetVelocity::Setup(ActionSetup& setup) { // Action_SetVelocity::DoAction() Action::RetType Action_SetVelocity::DoAction(int frameNum, ActionFrame& frm) { - std::copy( frm.Frm().xAddress(), frm.Frm().xAddress() + frm.Frm().size(), newFrame_.xAddress() ); - if (frm.Frm().HasVelocity()) - std::copy( frm.Frm().vAddress(), frm.Frm().vAddress() + frm.Frm().size(), - newFrame_.vAddress() ); + newFrame_.SetFrame( frm.Frm() ); if (mode_ == ZERO) { for (AtomMask::const_iterator atom = Mask_.begin(); atom != Mask_.end(); ++atom) { From 579c73850e5361d71628b18b00eeeeaa48118745 Mon Sep 17 00:00:00 2001 From: "Daniel R. Roe" Date: Thu, 4 Jun 2026 09:32:20 -0400 Subject: [PATCH 2/2] V7.9.0. Minor version bump for SetVelocity fix; all frame info now properly copied over. --- src/Version.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Version.h b/src/Version.h index 8f08c1d2be..8a4e4929fa 100644 --- a/src/Version.h +++ b/src/Version.h @@ -12,7 +12,7 @@ * Whenever a number that precedes is incremented, all subsequent * numbers should be reset to 0. */ -#define CPPTRAJ_INTERNAL_VERSION "V7.8.0" +#define CPPTRAJ_INTERNAL_VERSION "V7.9.0" /// PYTRAJ relies on this #define CPPTRAJ_VERSION_STRING CPPTRAJ_INTERNAL_VERSION #endif