UPGo-McGill / matchr

Fast and Reliable Matching of Images
Other
2 stars 0 forks source link

Inconsistent create_signature results with parallel processing #6

Open dwachsmuth opened 3 years ago

dwachsmuth commented 3 years ago

With plan(sequential), ceiling(sum(test_long_sig, na.rm = TRUE)) == 1034, but with plan(multisession) ceiling(sum(test_long_sig, na.rm = TRUE)) == 1035. Why? Some random seed, some subtle difference in the implementation?

dwachsmuth commented 3 years ago

There's also an inconsistency with (I think) the remove_black_bars argument on macOS vs. other platforms. For former, create_signature(urls[11]) gives aspect ratio 2.08, but for latter it's 2.04.