Results 340 comments of Bryce Mecum

Thanks. A `bt all` gives something similar. In an attempt to catch when the test was approaching timeout, I got a baseline for how long the test takes to complete...

I think a regression test would be enough, so long as you can reproduce the non-determinism in that test before making the code change.

Right. There might be a better way than this but adding a test that exercises `SchemaToFlatbuffer` and asserts the resulting Flatbuffer schema is byte-identical to some value you get, that...

Issue resolved by pull request 40392 https://github.com/apache/arrow/pull/40392

This has been merged. Thanks to those who reviewed and extra thanks to @noamross for contributing the initial PR.

Yeah, `guess_format_id` uses a hard-coded map between D1 format IDs and file extensions: https://github.com/NCEAS/arcticdatautils/blob/master/R/util.R#L79. I threw in a custom routine for NetCDF files that uses the metadata to guess the...

I honestly have never done this but I think your assessment of _how_ is right. For every S4 class constructor, a bare R function is provided which does three things:...

This'd be nice to see. A few of points: 1. A bag can have multiple payload manifests, one for each of the checksum algorithms used. So it's valid to have...

Oh right. I misinterpreted what I read. Appears to be the former: > o Every payload manifest MUST list every payload file name exactly :(

This is a good topic to bring up. > There is also some ambiguity over what the meaning of publisher is for these entries when the same data set can...