Is BasicType useful?

35 views
Skip to first unread message

Joachim Durchholz

unread,
Mar 10, 2015, 2:35:30 PM3/10/15
to sy...@googlegroups.com
Class BasicType defines no methods or attributes, its only subclass is
BasicMeta.
Neither class is in the sympy namespace.

Does BasicType serve any purpose that BasicMeta would not?

(Context: I found this while exploring SymPy's metaclass hierarchy.
Metaclass status should not affect the outcome of this question though.)

Aaron Meurer

unread,
Mar 10, 2015, 2:40:13 PM3/10/15
to sy...@googlegroups.com
I suppose it must be there to distinguish subtypes from other subtypes
of BasicMeta that aren't BasicType. Either that or it used to have
methods but they were deleted, and the class was not deleted. I would
look into the git history of it using git blame.

Aaron Meurer
> --
> You received this message because you are subscribed to the Google Groups
> "sympy" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to sympy+un...@googlegroups.com.
> To post to this group, send email to sy...@googlegroups.com.
> Visit this group at http://groups.google.com/group/sympy.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/sympy/54FF396B.4060808%40durchholz.org.
> For more options, visit https://groups.google.com/d/optout.

Joachim Durchholz

unread,
Mar 10, 2015, 3:16:45 PM3/10/15
to sy...@googlegroups.com
Am 10.03.2015 um 19:39 schrieb Aaron Meurer:
> I suppose it must be there to distinguish subtypes from other subtypes
> of BasicMeta that aren't BasicType.

No other subtypes exist.

> Either that or it used to have
> methods but they were deleted, and the class was not deleted. I would
> look into the git history of it using git blame.

It was created by Ondrej in ddd0349 (2007-09-23), as an empty superclass
to MetaBasicMeths.
It wasn't used as a superclass to anything else, not even then (but
MetaBasicMeths got renamed to BasicMeta along the way).
Maybe he anticipated that BasicType would acquire methods over time.

I'll try and see what happens if I remove it and change all references
to BasicMeta.

Aaron Meurer

unread,
Mar 10, 2015, 3:26:12 PM3/10/15
to sy...@googlegroups.com
I think this explains the issue
http://docs.sympy.org/latest/guide.html#functions (search for "meta").
By the way, I still think we should remove this document
https://github.com/sympy/sympy/pull/2807.

Aaron Meurer
> --
> You received this message because you are subscribed to the Google Groups
> "sympy" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to sympy+un...@googlegroups.com.
> To post to this group, send email to sy...@googlegroups.com.
> Visit this group at http://groups.google.com/group/sympy.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/sympy/54FF4319.40307%40durchholz.org.

Joachim Durchholz

unread,
Mar 10, 2015, 4:02:40 PM3/10/15
to sy...@googlegroups.com
Am 10.03.2015 um 20:25 schrieb Aaron Meurer:
> I think this explains the issue
> http://docs.sympy.org/latest/guide.html#functions (search for "meta").

That's outdated, MetaBasicMeths does not exist anymore (it might have
been removed in 2008 but I'm not 100% sure).

> By the way, I still think we should remove this document
> https://github.com/sympy/sympy/pull/2807.

Heh. I agree with "rather no docs than bad docs".
I have no idea who could work on salvaging what's still useful. But
maybe if there is nobody around who could or would maintain this, it's
better to get rid of it and reduce the maintenance burden.

Aaron Meurer

unread,
Mar 10, 2015, 4:10:46 PM3/10/15
to sy...@googlegroups.com
On Tue, Mar 10, 2015 at 3:02 PM, Joachim Durchholz <j...@durchholz.org> wrote:
> Am 10.03.2015 um 20:25 schrieb Aaron Meurer:
>>
>> I think this explains the issue
>> http://docs.sympy.org/latest/guide.html#functions (search for "meta").
>
>
> That's outdated, MetaBasicMeths does not exist anymore (it might have been
> removed in 2008 but I'm not 100% sure).

Right. So it's probably safe to remove the subtype.

>
>> By the way, I still think we should remove this document
>> https://github.com/sympy/sympy/pull/2807.
>
>
> Heh. I agree with "rather no docs than bad docs".
> I have no idea who could work on salvaging what's still useful. But maybe if
> there is nobody around who could or would maintain this, it's better to get
> rid of it and reduce the maintenance burden.

That was my opinion too but others disagreed so it wasn't merged.

Aaron Meurer

>
> --
> You received this message because you are subscribed to the Google Groups
> "sympy" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to sympy+un...@googlegroups.com.
> To post to this group, send email to sy...@googlegroups.com.
> Visit this group at http://groups.google.com/group/sympy.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/sympy/54FF4DD7.1090904%40durchholz.org.

Joachim Durchholz

unread,
Mar 10, 2015, 4:47:59 PM3/10/15
to sy...@googlegroups.com
Am 10.03.2015 um 21:10 schrieb Aaron Meurer:
>>> By the way, I still think we should remove this document
>>> https://github.com/sympy/sympy/pull/2807.
>>
>>
>> Heh. I agree with "rather no docs than bad docs".
>> I have no idea who could work on salvaging what's still useful. But maybe if
>> there is nobody around who could or would maintain this, it's better to get
>> rid of it and reduce the maintenance burden.
>
> That was my opinion too but others disagreed so it wasn't merged.

I see.
Adding my opinion.
Reply all
Reply to author
Forward
0 new messages