Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recognize the equivalence of circularity and crosslinks which generate circular molecules. #109

Open
paulflang opened this issue Oct 5, 2020 · 3 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@paulflang
Copy link
Contributor

The circular attribute seems redundant to crosslinking (at least for proteins). Not sure if we are OK with providing two ways to do the same thing.

@jonrkarr
Copy link
Member

jonrkarr commented Oct 5, 2020

Semantically, I think its useful to have a special syntactic sugar for circularity. Semantically, this connotes different meaning than a crosslink.

More generally, BpForms is not founded on the principle that there should only be 1 unique way to describe a polymer.

  • For circular polymers, there isn't necessarily a well-defined start position
  • There's nothing that prevents the community from defining monomeric forms which are themselves polymers. If such a monomeric form is semantically meaningful on its own and has a commonly accepted id or name, I don't see a problem with this.

The main goal of BpForms is to be concrete. I don't think this requires a unique representation of each molecule.

@paulflang
Copy link
Contributor Author

OK, thanks. This all makes sense.

For circular polymers, there isn't necessarily a well-defined start position

I noticed that this a problem where the bpforms equality check fails. I had a brief look at the code, but the way it is structured I couldn't find a trivial fix.

@jonrkarr
Copy link
Member

jonrkarr commented Oct 5, 2020

Yes, that a good point. We should change the code to recognize the equivalence of this special case. This comes up in a couple places

  • equality
  • it shouldn't be possible to describe a circular molecule with a crosslink between the same atoms; this should result in an error.

@jonrkarr jonrkarr changed the title The circular attribute is redundant to crosslinking Recognize the equivalence of circularity and crosslinks which generate circular molecules. Oct 5, 2020
@jonrkarr jonrkarr added bug Something isn't working enhancement New feature or request and removed discussion labels Oct 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants