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

Eliminate Long restrictions for protocol parameters #38

Open
bengtmartensson opened this issue Apr 11, 2017 · 2 comments

Comments

@bengtmartensson
Copy link
Owner

commented Apr 11, 2017

Presently, Number-s are restricted to 64 (or 63) bit length, due to the use of Java longs. Lift this restriction. Provide alternative (to the present) implementation of Number (and ??). Should dynamically decide on the usage of long and used long or BigInteger.

Idea: Extend the class Number with some member functions (arithmetic etc) of BigInteger. The functions in Numerical should return Number. The arithmetic presently in long should them be performed with the member functions of Number (not to be confused with java.lang.Number).

@bengtmartensson bengtmartensson added this to the Milestone 1 milestone Apr 11, 2017

@bengtmartensson bengtmartensson removed this from the Milestone 1 milestone Apr 26, 2017

@bengtmartensson

This comment has been minimized.

Copy link
Owner Author

commented Apr 26, 2017

Right now, it appears to be very much work for fairly little gain. When using the analyzer of very long signal, it (analyzer) can be taught to handle this restriction.

@bengtmartensson bengtmartensson added this to the Longtime milestone Apr 27, 2017

@bengtmartensson

This comment has been minimized.

Copy link
Owner Author

commented Jun 29, 2017

The analyzer is really a case or itself and probably can be fixed independently, using BigInteger.

bengtmartensson added a commit that referenced this issue Jul 14, 2017

@bengtmartensson bengtmartensson changed the title Eliminate Long restrictions Eliminate Long restrictions for protocol parameters Apr 2, 2018

bengtmartensson added a commit that referenced this issue Nov 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.