Use of pycutest with virtual environments about pycutest HOT 11 CLOSED

Thank you very much!

Regarding your issue, we do actually have a fallback in that PyCUTEst will default to using the current working directory to store the cache if the ${PYCUTEST_CACHE} environment variable is not defined (see the end of system_paths.py), although I realise that we have probably not documented this anywhere and we should really make this behaviour clear in the README and documentation.

from pycutest.

I see that default now, good to know! I'll have a bit of time in the next couple of weeks where I can probably document that and PR it.

What would you think of pycutest adding the contents of the ${PYCUTEST_CACHE} variable to the python path automatically? (or at least providing a function to do it automatically). Would you be interested in a PR doing that?

(sorry for the slow response btw)

from pycutest.

No worries, a documentation PR for the default cache handling would be much appreciated, many thanks!

Regarding pycutest automatically adding the contents of ${PYCUTEST_CACHE} to the python path, I will have to ask my co-author who wrote the environment variable handling code. @lindonroberts do you think this would be a good idea?

from pycutest.

I think adding PYCUTEST_CACHE to the Python path automatically would be a nice feature. This would simplify the installation a little bit too (only needing to set PYCUTEST_CACHE and not PYTHONPATH). Happy for a PR on this

from pycutest.

Because I had to deal with this today, a code snippet I needed to update the Python path at runtime to include PYCUTEST_CACHE:

import os, sys
try:
    sys.path.index(os.environ['PYCUTEST_CACHE'])
except ValueError:
    sys.path.append(os.environ['PYCUTEST_CACHE'])

In our case it would be similar, but with get_cache_path() instead of os.environ['PYCUTEST_CACHE'], so

import sys
try:
    sys.path.index(get_cache_path())
except ValueError:
    sys.path.append(get_cache_path())

It would make sense for this to go at the end of __init__.py, since that also does things like check existence of environment variables.

from pycutest.

Sounds good to me, would be great to have this feature if you're able to add it?

from pycutest.

Reopening as a reminder to mention the "if environment variable not available use the current directory" behavior in the documentation

from pycutest.

Many thanks! I guess we should probably think about a new release with updated docs soon?

from pycutest.

I agree - to this end in commit f0b5427 I have updated:

• Increased version number to 1.1
• Documentation to include simplified installation (remove any reference to updating PYTHONPATH), updated version history. I set the release date to 14 Feb to give @jfowkes time to check things over
• Misc documentation changes: updated index.rst to use my homepage rather than (old) email, updated link for Windows/WSL info

from pycutest.

Fantastic, thank you very much @lindonroberts for this addition and for the comprehensive docs update. This looks great to me, please release the new release!

from pycutest.

@jfowkes - I've tagged release v1.1 and made it available on pyPI (https://pypi.org/project/pycutest/)

from pycutest.