search

alexbol99 / flatten-interval-tree

Interval binary search tree
MIT License
41 stars 21 forks source link

Support for bigint #51

Open MarianoFacundoArch opened 5 months ago

MarianoFacundoArch commented 5 months ago

Can we adapt the code so that it supports bigint?

/@flatten-js/interval-tree/dist/main.mjs:155 this.item.key = new Interval(Math.min(key[0], key[1]), Math.max(key[0], key[1])); ^

TypeError: Cannot convert a BigInt value to a number at Math.min ()

alexbol99 commented 5 months ago

Issue resolved in v1.1.3 I changed usage of Math functions to simple expression @MarianoFacundoArch , please confirm if this is enough, I've never worked before with bigint numbers.