AssertionError:

Open
#8,775 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Bug
Clarity
Needs clarification
Activity status
Stale
Tech stack
android, python
Domain
databases, mobile

Research direction

Start by tracing removeLocalTx in electrum/gui/qml/qetxdetails.py through get_depending_transactions in address_synchronizer.py to get_spent_outpoints in wallet_db.py, using the reported traceback lines as entry points. Reproduce the removal operation with an imported wallet on Android and confirm it no longer raises the AssertionError.

Written by the indexing model from the issue text.

Description

how-to-reproduce?

Crash Report

This crash report was reported through the automatic crash reporting system 🤖

Traceback

Traceback (most recent call last):
  File "/home/user/wspace/electrum/.buildozer/android/app/electrum/gui/qml/qetxdetails.py", line 398, in removeLocalTx
  File "/home/user/wspace/electrum/.buildozer/android/app/electrum/address_synchronizer.py", line 412, in get_depending_transactions
  File "/home/user/wspace/electrum/.buildozer/android/app/electrum/json_db.py", line 44, in wrapper
  File "/home/user/wspace/electrum/.buildozer/android/app/electrum/wallet_db.py", line 1261, in get_spent_outpoints
AssertionError

AssertionError: 

Reporter

This issue was reported by 2 user(s):

Electrum Version Python Version Operating System Wallet Type Locale Stack
4.4.6 3.8.15 (default, Aug 17 2023, 15:47:24) [Clang 11.0.5 (https://android.googlesource.com/toolchain/llvm-project 87f1315d Android 11 on xiaomi ginkgo (RKQ1.201004.002 test-keys) imported en_US
4.4.6 3.8.15 (default, Aug 17 2023, 15:47:24) [Clang 11.0.5 (https://android.googlesource.com/toolchain/llvm-project 87f1315d Android 12 on samsung beyond1q (SP1A.210812.016.G973USQU7IVH3) imported en_US

Additional Information

The reporting user(s) did not provide additional information.

Dominant language
Python
Stars
8.6k
Forks
3.5k
Avg merge
2d 8h
Merged PRs (30d)
61

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from spesmilo/electrum

All issues in spesmilo/electrum

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.