Update calcTargetingDistance

Issue #77 resolved
Julian Zhou created an issue

See comment by @javh in this issue :

"And we should update shazam::calcTargetingDistance accordingly as well."

(@javh: I don't know who wrote this function; feel free to change the assignee)

Comments (6)

  1. Jason Vander Heiden

    Also, we'll need to update the distance matrices in changeo when we are done updating them in shazam, because changeo.DefineClones and shazam::distToNearest must use identical matrices.

  2. ssnn

    Are you going to fix distToNearest + tests? It is going to be more than updating the matrices names. I tried locally updating the h1sf matrix and the results are different, so this needs to be evaluated and changed as needed.

  3. Julian Zhou reporter

    @ssnn @javh : I think I've updated all that need to be updated in shazam, including the tests. The pipeline built successfully.

    @javh: As discussed, you will update the relevant distance matrices in CHANGE-O.

  4. Log in to comment