Skip to content
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

fix transfer of large data #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix transfer of large data #43

wants to merge 1 commit into from

Conversation

derekwisong
Copy link

Data larger than the max integer value would fail to transfer. changed
size of message read from the IPC protocol from an int to an unsigned
int. Also changed to load large data in chunks since stream.read() does
not support values larger than max int.

Data larger than the max integer value would fail to transfer. changed
size of message read from the IPC protocol from an int to an unsigned
int. Also changed to load large data in chunks since stream.read() does
not support values larger than max int.
@bartosz-dolecki
Copy link
Collaborator

Thank you for contributing code to the the Exxeleron GmbH open source projects! Before we can merge the changes you propose we need a contributor license agreement from you.

In brief, by submitting your code to a Exxeleron open source project, you are granting us a right to use that code under the terms of this Agreement, including providing it to others. You are also certifying that you wrote it, and that you are allowed to license it to us. You are not giving up your copyright in your work.

Contributor License Agreements are important because they define the chain of ownership of a piece of software. Some companies won’t allow the use of free software without clear agreements around code ownership. That’s why many open source projects collect similar agreements from contributors.

To sign the contributor license agreement, please log in to GitHub and follow the link contained in:
https://github.com/exxeleron/qPython/blob/master/CONTRIBUTING.md

Best regards,
Exxeleron team

@nugend
Copy link

nugend commented Feb 14, 2017

I think in Python 3, sys.maxsize is the appropriate limit for the representation. See here: http://stackoverflow.com/questions/13795758/what-is-sys-maxint-in-python-3

Actually, sys.maxsize might be appropriate for both Python 2 and 3?

@mkujbus
Copy link

mkujbus commented Jul 28, 2022

why is this not get merged?

@nugend
Copy link

nugend commented Jul 30, 2022

@mkujbus this repo has been abandoned. There is no public effort to maintain the qpython library, unfortunately.

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