<div dir="ltr"><div style="font-size:12.8px"><div><div><div><div>Hello,<br><br></div>My name is Vishnu and I'm working as a developer with skills in Java and Python. I am keen to contribute to open source projects and know about Biopython since when I was in college although I haven't used it much.<br><br></div>I was looking into issue #664 on github and went though the code. The issue is fixed but I have some doubts on this fix : <br><br></div>1. There is a change in regular expression that needs to be done in order to make it compatible to handle negative weights. Version 1.58 handles this is a different way, which is the preferred method?<br><br></div>2. The need for functional test cases was discussed in the page, should that be included too before I commit the code?<br><br></div><span style="font-size:12.8px">Thanks in advance!</span><div class="" style="font-size:12.8px"></div></div>