You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While evaluating DYAD with the workflow benchmark, we found that cmp_canonical_path_prefix had a surprisingly large cost in terms of time. Some fixes, as suggested by @hariharan-devarajan, are:
Remove the use of realpath
Switch to a prefix tree-based algorithm for path comparison
The text was updated successfully, but these errors were encountered:
You can cache the result of the first realpath() call as it translates the dyad managed path to real path.
The second one is there for correctness. You cannot omit that.
While evaluating DYAD with the workflow benchmark, we found that
cmp_canonical_path_prefix
had a surprisingly large cost in terms of time. Some fixes, as suggested by @hariharan-devarajan, are:realpath
The text was updated successfully, but these errors were encountered: