Tim Hobson

Hi @coffnix. I'm getting the same error reported above. After deleting my `wallet.dat` file I get this: ``` $ node dist/src/bitcoin.js Loading configuration from /etc/ion/testnet-bitcoin-config.json... Loading ION bitcoin versioning config...

In the DPTextDETR `Pipeline.ipynb` notebook, the line: ``` from mapreader.text_spot.dptext_detr_runner import DPTextDETRRunner ``` should be: ``` from mapreader.spot_text.dptext_detr_runner import DPTextDETRRunner ``` That line also failed initially (for me) because the...

Might be worth adding a line at the top of the part1 notebook to remind people to select the `Python (mr_py)` kernel. (Seems obvious, but I forgot first time.)

May require a new patch type, patch_rename, for renaming columns.

Running the unit tests I found that the existing test for `columnwise_candidates` fails because the `constraints` argument is missing. Is it possible to have a default argument to indicate "no...

### Synthetic experiment configuration Parameters: * data frame identifier (e.g. source & dataset name) * candidate datadiff function * corruption parameters consisting of a list of sample_patch_xxx closures appropriate for...

## Fidelity metrics ### Patch standard form It may be easier/quicker to test for the particular "standard" characteristics necessary for a given operation, rather than to have a general function...