search

djkoloski / rust_serialization_benchmark

Benchmarks for rust serialization frameworks
660 stars 55 forks source link

Readme updater just recategorized most serializers as zero-copy #91

Closed mumbleskates closed 5 hours ago

mumbleskates commented 5 hours ago

It looks like the last workflow just put almost every library into the "zero-copy" category where they have no data. :thinking:

mumbleskates commented 5 hours ago

actually... it looks like it mostly just swapped everything?

djkoloski commented 5 hours ago

Looks like #90 unintentionally changed the grouping logic in the README formatter. I committed a fix in 71cfd75515e6de4f97ddd8ad74738c814c8241a1 and kicked a new bench. It should be updated in an hour or so.

mumbleskates commented 5 hours ago

great, you got to the bottom of that just before i could. it looked like the logic got mostly swapped but not quite, i think you've got it right.