jams icon indicating copy to clipboard operation
jams copied to clipboard

Allow to change the default indentation of the produced JSON.

Open hendriks73 opened this issue 5 years ago • 3 comments

The default is 2, which can lead to fairly large bloat, when considering ten-thousands of JAMS (think global tempo estimates like https://github.com/tempoeval/tempo_eval/tree/main/annotations). Of course one can go the gzip/jamz route, which will create smaller files, but at the price of (easy/casual) editability.

Being able to simple pass indent=None is a compromise. It still allows manual editing, while also reducing size quite a bit.

hendriks73 avatar Oct 21 '20 12:10 hendriks73

Closed and re-opened PR to trigger Travis build. Since Master has changed, it should run through this time. The change is trivial.

hendriks73 avatar Nov 30 '20 10:11 hendriks73

Lgtm, @bmcfee any comments before I merge and bump the patch version?

justinsalamon avatar Mar 10 '21 01:03 justinsalamon

Lgtm, @bmcfee any comments before I merge and bump the patch version?

LGTM, thanks!

Before bumping the version, I think it's worth combing through open issues and seeing if there's anything else that needs fixing in the short term.

bmcfee avatar Mar 10 '21 14:03 bmcfee