remove external (non-core-python) library argparse as a dependency

[sugatoray]

Describe the bug

The current dependencies include installing an external library: argparse.

It must be noted that argparse is a part of the core python libraries. There is no need for installing it alike other external libraries.

• docs for core-library, argparse: https://docs.python.org/3/library/argparse.html

🔥 EDIT: The library (argparse) that you are installing from PyPI is no longer maintained as it is now a part of standard python3. See my comment here.

See further details:

• https://gitter.im/conda-forge/conda-forge.github.io?at=62598b5b0466b352a46afd25

pygod/requirements.txt

Lines 1 to 5 in d037b67

	argparse>=1.4.0
	numpy>=1.19.4
	scikit-learn>=0.22.1
	scipy>=1.5.2
	setuptools>=50.3.1.post20201107

[yzhao062]

thanks @sugatoray ! We are aware of this and suggest keeping it. Our next few releases will roll out some more usages by CLI, and argparse is needed in that scenario.

[sugatoray]

@yzhao062 So, this library argparse is kept by design then? I wondered if it was there by mistake.

What is the difference between this and core-python argparse?

Thank you.

[kayzliu]

@yzhao062 So, this library argparse is kept by design then? I wondered if it was there by mistake.

What is the difference between this and core-python argparse?

Thank you.

I see the problem. We will remove the argparse in requirement before next release. Thanks! 😊

[sugatoray]

The following is from the README file of the source of PyPI version of argparse library. Since argparse is now a part of core-python (since 2015), it is best to remove the argparse from the requirements.txt.

• source: https://github.com/ThomasWaldmann/argparse/

pygod/requirements.txt

Lines 1 to 5 in d037b67

	argparse>=1.4.0
	numpy>=1.19.4
	scikit-learn>=0.22.1
	scipy>=1.5.2
	setuptools>=50.3.1.post20201107