-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error listing upon suggested tests #48
Comments
It could be some strange anaconda issue. Wouldn't be the first time. |
Hi @jonesparg, thank you for reaching out! Did you run the commands from the root of directory of a clone of the repository? |
I did not clone the repository - at least that I am aware of. I'm not much
of a GitHub user. I just installed persistable locally on my computer,
then ran the tests. Does one have to clone the repository in order to get
everything to work?
Apologies - I do work in Python but am primarily an R programmer, coming
over from the statistics side of things. R is a bit of a different beast
in terms of installing packages and so forth.
…On Tue, Nov 28, 2023 at 11:20 AM LuisScoccola ***@***.***> wrote:
Hi @jonesparg <https://github.com/jonesparg>, thank you for reaching out!
Did you run the commands from the root of directory of a clone of the
repository?
—
Reply to this email directly, view it on GitHub
<#48 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFKLIKFMX7OLFNYNBCOO3HTYGYFOBAVCNFSM6AAAAAA76AOP2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZQGIZDEOBSG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Yes, right now the tests check that persistable can run on your computer (rather than checking if the version installed with pip is running properly). Thus, you need to clone the repository, and then run those commands from the root directory of the repository clone. Let me know if you encounter any issues. |
Right. First time I've cloned a repository and actually put it on my
computer. I'll give it a go.
It's never too late for an old statistician to learn new tricks.
…On Tue, Nov 28, 2023 at 1:55 PM LuisScoccola ***@***.***> wrote:
Yes, right now the tests check that persistable can run on your computer
(rather than checking if the version installed with pip is running
properly). Thus, you need to clone the repository, and then run those
commands from the root directory of the repository clone. Let me know if
you encounter any issues.
—
Reply to this email directly, view it on GitHub
<#48 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFKLIKCOH5BRWHROW5PP57TYGYXRDAVCNFSM6AAAAAA76AOP2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZQGQ4DOMBTGM>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Okay, that wasn't hard.
It all worked and I think I am set to go, even if I use the Anaconda
environment instead of VSCode which is generally more tractable for me.
…On Tue, Nov 28, 2023 at 2:01 PM L. E. Jones ***@***.***> wrote:
Right. First time I've cloned a repository and actually put it on my
computer. I'll give it a go.
It's never too late for an old statistician to learn new tricks.
On Tue, Nov 28, 2023 at 1:55 PM LuisScoccola ***@***.***>
wrote:
> Yes, right now the tests check that persistable can run on your computer
> (rather than checking if the version installed with pip is running
> properly). Thus, you need to clone the repository, and then run those
> commands from the root directory of the repository clone. Let me know if
> you encounter any issues.
>
> —
> Reply to this email directly, view it on GitHub
> <#48 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AFKLIKCOH5BRWHROW5PP57TYGYXRDAVCNFSM6AAAAAA76AOP2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZQGQ4DOMBTGM>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
|
These errors occurred when I ran the following tests shown on the GitHub page after installing persistable on the command line, which went without problems:
pip install pytest playwright pytest-playwright
python -m playwright install --with-deps
pip install -r requirements.txt
python -m setup build_ext --inplace
pytest .
As suggested, I'm sending them on. See attached screen grab.
The text was updated successfully, but these errors were encountered: