Module name
See original GitHub issueThis is just meant as a discussion. But something that probably would have to be decided before the release of v1.0
.
What about changing the module name from SimPEG
to simpeg
?
-
Pro’s:
-
Following the convention of most packages, in that the package and module names are identical.
-
Following pep8 guidelines, in which packages and modules should have all-lowercase names.
-
Less confusing. At the moment what a potential user would do (and what happend to me at the beginning several times) is
pip install simpeg
- Fire-up an IPython shell or a notebook
import simpeg
with the result that it throws an error stating that
simpeg
is not installed. One has then to check the documentation or the examples to find out that the module to import is calledSimPEG
, notsimpeg
.
-
-
Con’s: Maybe too late for a change, too many scripts and notebooks exist which import
SimPEG
.
Just thought I initiate once the discussion. Maybe it was discussed before, I don’t know.
Issue Analytics
- State:
- Created 5 years ago
- Comments:7 (7 by maintainers)
Top GitHub Comments
I think there is a possibility to deliver a new
simpeg
package and at the same time aSimPEG
package which will only print information about the name-change. The important bit would be the linein
setup.py
. Then,pip install simpeg
orpip install SimPEG
(pip
is case-insensitive, I believe) would install both packages,simpeg
andSimPEG
. The directorySimPEG
would then only contain a__init__.py
-file. In there, we could put print-statements about all the changes. E.g.:SimPEG->simpeg
,EM->electromagnetics
, etc etc.In this case, if someone does
import simpeg
all is good and as it used to be before withSimPEG
. However, if someone doesimport SimPEG
, it will print all the info we put into__init__.py
.Here an example with a fictitious
testme
/TestMe
-package, install them viapython setup.py install
: testme.zipAnd here a screenshot of it in action:
I am closing this. As it didn’t happen in the simulation-PR it is unlikely going to happen. Re-open if you disagree 😉