John Marshall

Results 273 comments of John Marshall

I'm glad all other problems are solved, so that we can focus on quibbling about the text describing something for which the intention is IMHO clear… :smile: The intention is...

As for newlines: SAM files are described as “text files”, so end-of-line indicators are (of course) whatever the host operating system conventions say they are.[^1] As a quality of implementation...

Hmmm… I think my previous two comments have somewhat contradicted each other, as the first comment was really based too much on my own worldview of Unix as the natural...

At last month's meeting, @daviesrob pointed out that the specification text “all fields are encoded using 7-bit US-ASCII” is intended to disallow UTF-16 et al and require the lowest common...

Apropos the service-info part of this (which replaces the older service-info response with the more-detailed JSON in [service-info as ratified by GA4GH](https://github.com/ga4gh-discovery/ga4gh-service-info)): TASC maintains a [list of the [service-info] type...

The recent merge accidentally checked in some merge conflict markers and multiple copies of text, which makes _refget.md_ in particular difficult to review. I have produced a version of this...

From https://github.com/samtools/hts-specs/pull/479#discussion_r786941363: > Shouldn't we take the opportunity of a major version change to move the service-info [to] the top level? I think this is what's implied in the [explanation](https://github.com/ga4gh-discovery/ga4gh-service-info#how-do-i-describe-a-service-implementing-multiple-specifications)...

That makes sense as an explanation, but I have not seen any expectation or hint in the spec that the endpoint might be or ought to be under `…/refget/…`. I've...

Probably yes to talking to Discovery, though in my (small) experience I have not had much luck in getting specific answers to practical questions of this sort… 🙃 I thought...

@mcupak: This is precisely the scenario that I asked about nearly three years ago. Htsget serves reads (BAM/CRAM files) and variants (VCF files), both of which are used in conjunction...