Default amount of gas is too small about near-api-js HOT 6 CLOSED

problem

unable to complete cross contract call tutorial due to upper limit on size of number that can be passed to a smart contract function call via nearlib

detail

recently @evgenykuzyakov submitted to Slack #general (see line 72 in main.js) an example of increasing the amount of gas submitted to 9.9 NEAR when invoking a function call. This seems to work find as a passthrough value since no checks are made in nearlib on the size of the number, it goes directly to nearcore. The example works fine.

But since we're using BN.js in nearlib we're limited by this assertion:
assert(number < 0x20000000000000); // 2 ^ 53 (unsafe)

This line limits us to
9007199254740991, which is much smaller than
99000000000000000000 suggested by the working example linked above.

Transaction 6fci1ciicWLM7JFcdZVDWaDXtJpzMScvvA6GmEuLPTZc failed. Exceeded the prepaid gas

reproduction

• open https://studio.nearprotocol.com/?f=7cd5vk11b

• run the test

• note error Transaction failed. Exceeded the prepaid gas.

• edit the value in src/test.js line 36 by incrementing the gas passed as an argument to contract.calculate() from
  9007199254740991 to
  9007199254740992

• run the test

• note error Error: Assertion failed coming from the assertion on this line in BN.js (one of our nearlib dependencies, line 3204)

Error: Assertion failed
    at assert (https://cdn.jsdelivr.net/npm/[email protected]/dist/nearlib.js:3082:21)
    at BN._initNumber (https://cdn.jsdelivr.net/npm/[email protected]/dist/nearlib.js:3204:7)

from near-api-js.

@bowenwang1996 Is the max amount of gas allowed for function calls a constant?

from near-api-js.

yes

from near-api-js.

@amgando

But since we're using BN.js in nearlib we're limited by this assertion:
assert(number < 0x20000000000000); // 2 ^ 53 (unsafe)

we aren't, just need to pass number as decimal string, i.e. new BN("99000000000000000000")

from near-api-js.

we aren't, just need to pass number as decimal string, i.e. new BN("99000000000000000000")

@vgrichina oh jeez 🤦‍♂

from near-api-js.

@vgrichina -- just thinking about this again now. BN is available in a JS context but this number is being passed to ContractPromise.create in an AssemblyScript context. unfortunately i'll have to revisit this more carefully after retracing my steps since I couldn't easily reproduce the problem I experienced before

from near-api-js.