KeyError: -1

Open
#5,212 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
35/100
Issue type
Bug
Clarity
Needs clarification
Activity status
Stale
Tech stack
python
Domain
desktop

Research direction

Start with the traceback in electrum/gui/qt/util.py, electrum/gui/qt/history_list.py, and electrum/util.py, especially the referenced handlers and value_from_pos. Reproduce the crash after changing a transaction description and trace the coordinate passed into value_from_pos. Done means that this action no longer raises KeyError: -1.

Written by the indexing model from the issue text.

Description

bug 🐞 Qt ✨

Crash Report

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

Traceback

  File "electrum/gui/qt/util.py", line 414, in on_commitData
  File "electrum/gui/qt/history_list.py", line 692, in text_txid_from_coordinate
  File "electrum/util.py", line 1068, in value_from_pos

KeyError: -1

Reporter

This issue was reported by 1 user(s):

Electrum Version Python Version Operating System Wallet Type Locale Stack
3.3.4 3.6.4 (default, Feb 28 2018, 03:24:19) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.42.1)] Darwin-18.2.0-x86_64-i386-64bit standard ? ℹ

Additional Information

changed the description of a transaction.

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.