argotorg / argotorg/fe

Replace num_bigint with primitive_types::U256?

Open
#414 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
1.7k
Forks
218
Avg merge
1d 7h
Merged PRs (30d)
6

Description

### What is wrong?

From #410:

We may need to reconsider the usage of BigInt,
I think it would be better to use U256 because:

1. it provides more reliability than multiple-precision arithmetic.
2. it improves performance a little bit.

### How can it be fixed

@Y-Nak? I don't really have any feelings about this; just copied it here so it doesn't get lost in the closed PR.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the current BigInt usage and the context in issue #410, then compare it with primitive_types::U256. Determine the full migration scope and compatibility implications before changing anything; done means the project consistently uses U256 where intended and its existing behavior remains covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
blockchain
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.