-
assigned issue to
Update calcTargetingDistance
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)
-
-
Also, we'll need to update the distance matrices in changeo when we are done updating them in shazam, because
changeo.DefineClones
andshazam::distToNearest
must use identical matrices. -
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.
-
reporter @ssnn I'll look into it this afternoon. Need to write some new functions first.
-
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.
-
reporter - changed status to resolved
- Log in to comment