Python 2 support about bayespy HOT 3 CLOSED

Okay, I did some digging, and my proposed option to put argv=None would not be correct.
One would have to implement something like:

def parse_command_line_arguments(mandatory_args, *optional_args_list, **kwargs):
    argv = kwargs.get('argv', None)

I implemented this in my own fork of the code, but now I notice, this has not been the only place positional arguments have been used like that in the code. For instance, it is also used here, and in a couple other places in the package.

From this, I guess Python 2 support is not on the roadmap for the package, is it?

from bayespy.

Yep, Python 2 isn't supported unfortunately. BayesPy will raise an error in the future if one tries to install it with Python 2: 3fbb8d5 . It will be in the next release probably.

I chose not to support Python 2 because I liked some features in Python 3, and wanted to avoid doing the transition in the near future and needing to support two versions of Pythons. This same "bug" has been reported so many times that maybe it was a wrong choice. Anyway, I hope the installation error will at least avoid some confusion because the problem is then clearly stated.

from bayespy.

It all makes sense. In this case, I think preventing installation to complete for Python 2 is indeed the way to go.

from bayespy.