Skip to content
New issue

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

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

Already on GitHub? # to your account

Try georust #107

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Try georust #107

wants to merge 1 commit into from

Conversation

ringsaturn
Copy link
Owner

According to ringsaturn/geometry-rs#17 GeoRust suppose to be faster.

However in tzf-rs' benchmark, it cause much slower. Need to check.

@ringsaturn ringsaturn force-pushed the ringsaturn/try-georust branch from 6b52a05 to 1230148 Compare February 13, 2025 14:34
@ringsaturn
Copy link
Owner Author

2025-02-13 update

This branch:

➜  tzf-rs git:(ringsaturn/try-georust) ✗ cargo bench
   Compiling tzf-rs v0.4.10 (/Users/ringsaturn/Projects/tzf-rs)
    Finished `bench` profile [optimized] target(s) in 16.57s
     Running unittests src/lib.rs (target/release/deps/tzf_rs-42ae8ec8e7a6f75d)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running unittests src/bin/tzf.rs (target/release/deps/tzf-a4becdb1389a5699)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running benches/finders.rs (target/release/deps/finders-65a5d56ffff55e1c)
Finders/Default/0       time:   [43.568 µs 44.304 µs 45.060 µs]
                        change: [+3516.0% +3596.9% +3678.8%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 6 outliers among 100 measurements (6.00%)
  1 (1.00%) low severe
  2 (2.00%) low mild
  2 (2.00%) high mild
  1 (1.00%) high severe
Finders/Finder/0        time:   [270.93 µs 273.07 µs 275.18 µs]
                        change: [+5000.7% +5066.7% +5128.5%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 6 outliers among 100 measurements (6.00%)
  1 (1.00%) low severe
  1 (1.00%) low mild
  4 (4.00%) high mild

main branch:

➜  tzf-rs git:(main) ✗ cargo bench
   Compiling num-traits v0.2.19
   Compiling tzf-rs v0.4.10 (/Users/ringsaturn/Projects/tzf-rs)
   Compiling pprof v0.14.0
   Compiling plotters v0.3.7
   Compiling criterion v0.5.1
    Finished `bench` profile [optimized] target(s) in 17.04s
     Running unittests src/lib.rs (target/release/deps/tzf_rs-d36dc1f92330f59c)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running unittests src/bin/tzf.rs (target/release/deps/tzf-f35e745af6530e8a)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running benches/finders.rs (target/release/deps/finders-a85868609910774d)
Finders/Default/0       time:   [1.1777 µs 1.1873 µs 1.1975 µs]
                        change: [-97.383% -97.323% -97.261%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
  4 (4.00%) high mild
  1 (1.00%) high severe
Finders/Finder/0        time:   [3.9526 µs 3.9681 µs 3.9856 µs]
                        change: [-98.567% -98.549% -98.530%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 7 outliers among 100 measurements (7.00%)
  1 (1.00%) low severe
  1 (1.00%) low mild
  5 (5.00%) high mild

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant