Skip to content

Conversation

@RandomCraftr
Copy link

Regarding ticket , I propose the minimal change: every time type is checked against numpy.int64, I add a or statement to accept if numpy.int32.

(It appears that even python on Windows 64bits platform encode numpy.array by default in 32 bits.)

@dawndown
Copy link

when I run the python code in Windows system, it will report an error: AssertionError: scale values must all be integers. But when I run the code in Linux system, it works!!! So I suggest making some adjustments that the code could run in Windows platform. Thanks.

@mmarcatili
Copy link

I report the same issue - "AssertionError: scale must all be integers" - on Windows. I am unable to try this on Linux (I don't have a Linux system).

@mmarcatili
Copy link

I have now tried the modification suggested by RandomCraftr and can confirm that it seems to fix the problem on Windows.

Copy link

@mmarcatili mmarcatili left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes worked for me in Windows running Python 3.11.7

@dalya
Copy link
Owner

dalya commented Jun 10, 2024

Hi everyone, the pip install leads to an older version of the code. If instead you download the repository from github, you should be able to run the code without errors.

@mmarcatili
Copy link

Thanks Dalya,
I can confirm I installed from github using the directions here. I did also try a pip install and yes, it had other issues beyond those mentioned here.

@dalya
Copy link
Owner

dalya commented Jun 10, 2024

I see now. I will go over the code and add the option of numpy.int32 in places where an integer is required. It would take me a few days to get to that. Thank you for letting me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants