kwhopper

Results 55 comments of kwhopper

Here's a very basic run with deterministic offsets: [RASBenchmark.zip](https://github.com/drewnoakes/metadata-extractor-dotnet/files/4673863/RASBenchmark.zip) Results indicate the List version is ~35-40% "faster": ``` BenchmarkDotNet=v0.10.12, OS=Windows 10.0.18363 Intel Core i7-8700K CPU 3.70GHz (Coffee Lake), 1 CPU,...

Pushed a few code changes to address some benchmark differences. A few may be reverted but they are at least documented in a commit.

I updated BenchmarkDotNet to 0.12.1 in the next tests. Here's a new benchmark file that includes the List version of RAS, and similar calls for IndexedCapturingReader and IndexedSeekingReader: [RASBenchmark.zip](https://github.com/drewnoakes/metadata-extractor-dotnet/files/4675045/RASBenchmark.zip) With...

Pushed and added some of what you mentioned. The memory allocations are intriguing but not sure what to make of them without more testing: ``` BenchmarkDotNet=v0.12.1, OS=Windows 10.0.18363.836 (1909/November2018Update/19H2) Intel...

Here's the memory when RAS is always instantiated during a benchmark run: ``` BenchmarkDotNet=v0.12.1, OS=Windows 10.0.18363.836 (1909/November2018Update/19H2) Intel Core i7-8700K CPU 3.70GHz (Coffee Lake), 1 CPU, 12 logical and 6...

To sum up a little as you look at this... We know that byte arrays will always be read and then sliced and diced as metadata is parsed. The goal...

Thanks @drewnoakes. I've hounded you plenty the last few years so it's enough that you're still open to the idea. Parallel libraries isn't out of the question entirely to benchmark....

> Without implementing parallel versions of the library in terms of old/new it won't be easy to see a more realistic set of values. I'm happy to do this if...

If the makernote is stored as part of a standard TIFF-formatted file, then that's the correct class. I assume it would go through the exif classes but haven't looked into...

ToDo: ICC data spread across multiple JPEG segments is not currently implemented. It could be done with a concatenated array but will attempt to do it with ReaderInfo *Update -...