Re: [Biopython] Atom inheritance from Entity

João Rodrigues <[email protected]>
Newsgroups gmane.comp.python.bio.general
Message-ID <CAB=-b2YRXq2UcDqZHHo1R5Te3TFs321Pu7RH_OETnC+AVGBgWA@mail.gmail.com>
Wouldn't it be weird for an Atom to have a get_children method? I'm
somewhat more fine with a None attribute than a useless method.

A quarta, 4/09/2019, 09:08, Spencer Bliven <[email protected]>
escreveu:

> Entity also provides methods for managing ids, transforming atoms, etc. My
> preference would be for everything to inherit from Entity and just have 0
> children. This is also consistent with Structure, which is an Entity with
> 'None' parent.
>
> -Spencer
>
>
> On 4 September 2019 at 08:02:19, João Rodrigues (
> [email protected]) wrote:
>
> Hi Spencer,
>
> Something that bugged me in the past as well. It would be nicer to have
> everyone inherit from a single point, but since Atoms don't have children,
> it would be confusing to have these methods there. My understanding is that
> Entity only exists to provide methods to handle iteration over children
> objects. It's a compromise?
>
> Thanks for the efforts in typing the module!
>
> João
>
> Spencer Bliven <[email protected]> escreveu no dia terça,
> 3/09/2019 à(s) 22:53:
>
>> I was wondering why Atom doesn't inherit from Entity. It doesn't have
>> children, but it seems like it would be more consistent to have the whole
>> Structure stack inherit from a single point. Is anyone aware of the history
>> there?
>>
>> I ask because I'm trying to add type annotations to Bio.PDB, and mypy
>> flagged a number of places where my initial 'Entity' type conflicted with
>> Atoms. We can certainly get around this by accepting either Entity or Atom,
>> but I'm not sure that was intended.
>>
>> -Spencer
>>
>> _______________________________________________
>> Biopython mailing list  -  [email protected]
>> https://mailman.open-bio.org/mailman/listinfo/biopython
>
>

_______________________________________________
Biopython mailing list  -  [email protected]
https://mailman.open-bio.org/mailman/listinfo/biopython
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.