Antoine Poinsot

Results 549 comments of Antoine Poinsot

Thanks! Unfortunately CI is now failing on Coincurve installation: ``` Installed /opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/bip32-4.0-py3.8.egg Processing dependencies for bip32==4.0 Searching for coincurve=15.0 Reading https://pypi.org/simple/coincurve/ Downloading https://files.pythonhosted.org/packages/6f/a2/f2a38eb05b747ed3e54e1be33be339d4a14c1f5cc6a6e2b342b5e8160d51/coincurve-21.0.0.tar.gz#sha256=8b37ce4265a82bebf0e796e21a769e56fdbf8420411ccbe3fafee4ed75b6a6e5 Best match: coincurve 21.0.0 Processing coincurve-21.0.0.tar.gz error:...

I don't think this is related to 3.8. It also fails on the 3.10 job, and i suspect it would fail on all of them given enough time.

Unfortunate that the CI doesn't run automatically anymore, just kicked it let's see.

Hi, just letting you know i did not forget about this.

Fair enough, i'll get to it and i think i'll also drop the upper bound. -------- Original Message -------- On Thursday, 11/06/25 at 08:26 Pavol Rusnak ***@***.***> wrote: > prusnak...

I cherry-picked your second commit in a PR dedicated to properly move the project away from our deprecated ways: #49. I plan to make a first release for the project...

I have a separate PR which supersedes this one to upgrade to Coincurve. Unfortunately it seems to fail CI because of https://github.com/ofek/coincurve/issues/187 at the moment. I'll get a release PR...

> I'll let others weigh in For what it's worth i think ti's useful to have a BIP for this. > BIPs are for standardization across implementations. Policy is an...

A regression test would demonstrate whether the fix is necessary. :shrug:

> EDIT: Related to this, we should probably modify how the two iterators are chained together so that we only include spending coins from the DB if they're not part...