Change `./code/` to `./src/` about covpol HOT 8 CLOSED

Exactly!

I would generally default to having my scripts in a directory called src/, but you could also give it a different a name (I think technically this is supposed to be good practice - https://docs.python-guide.org/writing/structure/).

from covpol.

+1 for giving the directory a package-like name. This may be one of the reason I was having problems with importing libraries things in the tests (#14)

How about covid or covpol?

from covpol.

+1 for giving the directory a package-like name. This may be one of the reason I was having problems with importing libraries things in the tests (#14)
How about covid or covpol?

do you refer the whole repository or just the folder with the code? I think src for scripts makes sense? But happy to name the whole repository covpol?

Just the folder with the code. Then the code becomes a 'package'. The advantage to this is that you can import it from elsewhere (like in the tests) with:

from covpol.agent_classs import CountryAgent

rather than

from src.agent_classs import CountryAgent

from covpol.

done

from covpol.

hmm oh never had that ever. but okay, i think with the relative paths now that should not be an issue, should it?

from covpol.

Exactly!

I would generally default to having my scripts in a directory called src/, but you could also give it a different a name (I think technically this is supposed to be good practice - https://docs.python-guide.org/writing/structure/).

okay. I did not know that either :)

from covpol.

+1 for giving the directory a package-like name. This may be one of the reason I was having problems with importing libraries things in the tests (#14)

How about covid or covpol?

do you refer the whole repository or just the folder with the code? I think src for scripts makes sense? But happy to name the whole repository covpol?

from covpol.

+1 for giving the directory a package-like name. This may be one of the reason I was having problems with importing libraries things in the tests (#14)

How about covid or covpol?

My vote would be for covpol of those two.

from covpol.