vrs-python
vrs-python copied to clipboard
Replace ndjson with jsonl
@ahwagner mentioned he called newline-delimited JSON ndjson, but he actually meant to say JSON Lines . They appear to be very similar formats (ndjson allows blank lines), but we should replace ndjson with jsonl.
Thanks @korikuzma
I believe -- though I could be wrong -- that we are already in conformance with either. We would just want to change language and that should be enough.
We could make a --jsonl-out work as the current --ndjson-out does, with records separated by a single \n char, and make a new --ndjson-out which separates records by a random number of \n chars between 1 and 5, in order to make it clear that they are different and convince people to use jsonl over ndjson.
https://github.com/ga4gh/vrs-python/blob/9cbb4b610f4385d6cbef6872a62ca0809a1e38f7/src/ga4gh/vrs/extras/annotator/cli.py#L123
@theferrit32 only 5? dream bigger