Including phosphorus patterns?
maxentile opened this issue · comments
Hi @SimonBoothroyd !
Quick question: I was curious why these phosphorus patterns are commented out in the conversion script:
openff-recharge/scripts/convert-am1-bcc/convert.py
Lines 130 to 133 in 2a10e8c
Do you think it would be fine to uncomment these, or is there a pitfall to be aware of? (I didn't see a prior discussion of this in a brief search of the repository.)
Best regards,
Josh
Hey @maxentile,
I'm sorry for the delayed response - I've been preparing to start a new job so things have been pretty busy!
I was curious why these phosphorus patterns are commented out in the conversion script:
I simply hadn't had enough time to properly port these into SMIRKS just yet and ensure they are fully tested!
The easy bit is mapping the phosphorous codes into SMIRKS patterns which is the lines commented out. The trickier ( / tedious) bit is then updating the other codes (BCCs involving a 'dative' bond, nitrogen with a highly delocalized lone pair etc) to work alongside them.
Do you think it would be fine to uncomment these, or is there a pitfall to be aware of?
If you're refitting all the charges then you should be fine minus any chemical perception issues. If you need these to match closely to the original AM1BCC model or the canonical OE implementation of it however then it's likely that just uncommenting these lines won't be enough, and the other SMIRKS will need a bit of tweaking!
Thanks @SimonBoothroyd for the context and advice!
This all makes sense. Will proceed with caution about the issues you mention if porting these in the future.