Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use new deku writer #294

Closed
wants to merge 5 commits into from
Closed

Use new deku writer #294

wants to merge 5 commits into from

Conversation

wcampbell0x2a
Copy link
Owner

@wcampbell0x2a wcampbell0x2a commented Oct 19, 2023

No description provided.

@github-actions
Copy link

Benchmark for 49bff2a

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 7.0±0.00ms 5.6±0.00ms -20.00%
only_read/tplink_ax1800 27.7±0.05ms 20.5±0.01ms -25.99%
write_read/netgear_ax6100v2 1789.7±4.82ms 1795.6±10.18ms +0.33%
write_read/tplink_ax1800 9.9±0.03s 9.9±0.04s 0.00%

@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

Attention: 38 lines in your changes are missing coverage. Please review.

Comparison is base (f206fef) 73.05% compared to head (0eddff9) 73.31%.

❗ Current head 0eddff9 differs from pull request most recent head b877c67. Consider uploading reports for the commit b877c67 to get more accurate results

Files Patch % Lines
backhand/src/filesystem/writer.rs 47.36% 5 Missing and 15 partials ⚠️
backhand/src/metadata.rs 50.00% 0 Missing and 8 partials ⚠️
backhand/src/reader.rs 82.92% 5 Missing and 2 partials ⚠️
backhand/src/squashfs.rs 87.50% 1 Missing and 1 partial ⚠️
backhand-cli/src/bin/add.rs 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #294      +/-   ##
==========================================
+ Coverage   73.05%   73.31%   +0.25%     
==========================================
  Files          21       21              
  Lines        2913     2930      +17     
  Branches     2913     2930      +17     
==========================================
+ Hits         2128     2148      +20     
- Misses        519      520       +1     
+ Partials      266      262       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link

Benchmark for 49bff2a

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 7.0±0.00ms 5.6±0.01ms -20.00%
only_read/tplink_ax1800 27.7±0.06ms 20.5±0.07ms -25.99%
write_read/netgear_ax6100v2 1799.1±5.17ms 1825.8±11.80ms +1.48%
write_read/tplink_ax1800 10.1±0.08s 10.1±0.05s 0.00%

@wcampbell0x2a
Copy link
Owner Author

(looks like a leftover println is causing the performance effects)

@github-actions
Copy link

Benchmark for 7ba92bd

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 7.0±0.23ms 7.0±0.09ms 0.00%
only_read/tplink_ax1800 25.6±0.52ms 25.6±0.34ms 0.00%
write_read/netgear_ax6100v2 2.3±0.04s 2.3±0.05s 0.00%
write_read/tplink_ax1800 12.5±0.06s 12.5±0.13s 0.00%

@github-actions
Copy link

Benchmark for c69bfd4

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 8.4±0.00ms 6.7±0.00ms -20.24%
only_read/tplink_ax1800 33.1±0.04ms 24.6±0.09ms -25.68%
write_read/netgear_ax6100v2 2.2±0.01s 2.2±0.02s 0.00%
write_read/tplink_ax1800 12.1±0.06s 12.1±0.05s 0.00%

Copy link

Benchmark for c592038

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 5.4±0.01ms 5.4±0.01ms 0.00%
only_read/tplink_ax1800 21.7±0.06ms 21.8±0.08ms +0.46%
write_read/netgear_ax6100v2 1322.9±12.31ms 1295.6±5.36ms -2.06%
write_read/tplink_ax1800 7.4±0.18s 7.2±0.10s -2.70%

@wcampbell0x2a wcampbell0x2a changed the base branch from deku-update to master December 15, 2023 04:53
@github-actions github-actions bot added A-add backhand-cli: add A-replace backhand-cli: replace A-backhand-lib A-CI GitHub Action A-tests bakchand-tests A-backhand-cli labels Dec 15, 2023
@wcampbell0x2a wcampbell0x2a force-pushed the use-deku-writer2 branch 3 times, most recently from b877c67 to c49bec3 Compare December 16, 2023 20:00
@wcampbell0x2a wcampbell0x2a force-pushed the use-deku-writer2 branch 3 times, most recently from cedc675 to 7f28a84 Compare December 18, 2023 02:52
Copy link

Benchmark for d7635ec

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 5.6±0.01ms 5.4±0.00ms -3.57%
only_read/tplink_ax1800 21.9±0.03ms 21.4±0.02ms -2.28%
write_read/netgear_ax6100v2 1287.9±4.45ms 1289.0±9.24ms +0.09%
write_read/tplink_ax1800 7.1±0.01s 7.1±0.01s 0.00%

@wcampbell0x2a wcampbell0x2a force-pushed the use-deku-writer2 branch 2 times, most recently from d2fb374 to 1dea596 Compare December 29, 2023 00:24
Copy link

Benchmark for 1afb005

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 5.6±0.01ms 5.4±0.00ms -3.57%
only_read/tplink_ax1800 21.9±0.02ms 21.4±0.01ms -2.28%
write_read/netgear_ax6100v2 1290.8±4.61ms 1283.1±1.16ms -0.60%
write_read/tplink_ax1800 7.1±0.04s 7.1±0.02s 0.00%

Copy link

Benchmark for f8c6ef0

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 5.6±0.01ms 5.4±0.01ms -3.57%
only_read/tplink_ax1800 21.8±0.03ms 21.3±0.01ms -2.29%
write_read/netgear_ax6100v2 1284.6±2.79ms 1289.1±10.35ms +0.35%
write_read/tplink_ax1800 7.1±0.01s 7.0±0.01s -1.41%

Copy link

Benchmark for e80451a

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 5.7±0.00ms 5.1±0.00ms -10.53%
only_read/tplink_ax1800 21.9±0.06ms 20.7±0.01ms -5.48%
write_read/netgear_ax6100v2 1284.0±2.52ms 1284.6±3.96ms +0.05%
write_read/tplink_ax1800 7.0±0.01s 7.0±0.00s 0.00%

Copy link

Benchmark for 71dd8ce

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 5.6±0.12ms 5.1±0.00ms -8.93%
only_read/tplink_ax1800 21.7±0.05ms 20.8±0.35ms -4.15%
unsquashfs/full 12.6±0.04ms 12.6±0.03ms 0.00%
unsquashfs/full-path-filter 9.3±0.02ms 9.3±0.03ms 0.00%
unsquashfs/list 10.5±0.02ms 10.5±0.20ms 0.00%
unsquashfs/list-path-filter 9.6±0.09ms 9.6±0.02ms 0.00%
write_read/netgear_ax6100v2 1288.8±3.20ms 1283.6±2.55ms -0.40%
write_read/tplink_ax1800 7.1±0.02s 7.1±0.02s 0.00%

Copy link

Benchmark for 79378f8

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 5.6±0.01ms 5.1±0.01ms -8.93%
only_read/tplink_ax1800 22.3±0.18ms 20.9±0.12ms -6.28%
unsquashfs/full 13.9±0.38ms 13.3±0.22ms -4.32%
unsquashfs/full-path-filter 10.5±0.17ms 9.9±0.17ms -5.71%
unsquashfs/list 11.5±0.19ms 11.0±0.11ms -4.35%
unsquashfs/list-path-filter 10.5±0.13ms 10.0±0.07ms -4.76%
write_read/netgear_ax6100v2 1333.1±27.71ms 1326.7±27.72ms -0.48%
write_read/tplink_ax1800 7.5±0.08s 7.5±0.04s 0.00%

Copy link

Benchmark for a6563d6

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 2.8±0.00ms 2.5±0.00ms -10.71%
only_read/tplink_ax1800 7.2±0.01ms 6.7±0.01ms -6.94%
unsquashfs/full 10.4±0.23ms 10.5±0.26ms +0.96%
unsquashfs/full-path-filter 7.1±0.09ms 6.9±0.10ms -2.82%
unsquashfs/list 8.3±0.16ms 8.1±0.11ms -2.41%
unsquashfs/list-path-filter 7.4±0.10ms 7.2±0.10ms -2.70%
write_read/netgear_ax6100v2 1293.8±9.43ms 1288.0±3.40ms -0.45%
write_read/tplink_ax1800 7.1±0.01s 7.1±0.02s 0.00%

Copy link

Benchmark for 62733d9

Click to view benchmark
Test Base PR %
only_read/netgear_ax6100v2 2.8±0.00ms 2.6±0.00ms -7.14%
only_read/tplink_ax1800 7.2±0.01ms 6.7±0.01ms -6.94%
unsquashfs/full 10.5±0.14ms 10.1±0.18ms -3.81%
unsquashfs/full-path-filter 7.0±0.05ms 6.9±0.06ms -1.43%
unsquashfs/list 8.2±0.06ms 8.0±0.06ms -2.44%
unsquashfs/list-path-filter 7.3±0.05ms 7.2±0.09ms -1.37%
write_read/netgear_ax6100v2 1283.8±5.77ms 1278.5±3.00ms -0.41%
write_read/tplink_ax1800 7.1±0.02s 7.1±0.03s 0.00%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-add backhand-cli: add A-backhand-cli A-backhand-lib A-CI GitHub Action A-replace backhand-cli: replace A-tests bakchand-tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants