[AstroPy] Coding Guidelines draft (comments encouraged)
embray at stsci.edu
Mon Aug 15 10:12:13 EDT 2011
On 08/15/2011 02:30 AM, Erik Tollerud wrote:
>> I know people have been busy, myself included, but I wonder if anyone
>> has had time to consider the use of super() and the defense thereof.
> Based on the lack of further discussion on the top, it would seem not :)
>> I would change the coding guidelines read opposite of how they do
>> currently: Use of super() should be encouraged except where one knows
>> for sure that it's not going to be appropriate, due to the need for a
>> customized method resolution order. What *should* be discouraged is the
>> use of multiple inheritance except in the cases of orthogonal bases, or
>> if one is really sure what they're doing.
> You raise some very good points - my main concern was that it always
> be consistently super() or consistently not . I can see that use of
> super() would be a safer guideline given these use cases - does anyone
> object to this?
> If not (within a few days,hopefully), I will update the guidelines to
> suggest using super() unless there is a specific reason not to (and
> require that it be specifically mentioned if so), as well as
> discouraging multiple inheritance without a good reason (and replace
> the old example with an orthogonal base/mixin example that uses
Thanks! Sounds like a good plan. I may also add an example of my own,
but a mixin example sounds like a good start.
>> On an unrelated matter, I have one other suggestion for the coding
>> A top level package's __init__.py should be importable under all
>> circumstances, and should not too much "stuff" in it, for lack of a
>> better word. To explain: Several of our own packages have an
>> __init__.py that contains a lot of functionality, such that when I try
>> to import it, it might crash (due to some prerequisite not being met),
>> take a very long time (due to it importing many submodules
>> unnecessarily), or it might have actual side effects like writing to the
> Sounds good to me - feel free to write up such a note and submit it
> either in github pull request form or just on this mailing list.
Ah, I hadn't realized that the coding guide and other docs were on
github now. I will do that then. Thanks again,
More information about the AstroPy