samepath: speed optimize

Author: scivision

+stdlib/+python/samepath.m

@@ -1,17 +1,10 @@
 function y = samepath(path1, path2)
-arguments
-  path1 (1,1) string
-  path2 (1,1) string
-end
 
 try
   y = py.os.path.samefile(path1, path2);
 catch e
-  if contains(e.message, "FileNotFoundError")
-    y = false;
-  else
-    rethrow(e);
-  end
+  pythonException(e)
+  y = false;
 end
 
 end

+stdlib/samepath.m

@@ -20,11 +20,9 @@
 arguments
   path1 string
   path2 string
-  backend (1,:) string = ["python", "perl", "java", "sys", "native"]
+  backend (1,:) string = ["python", "java", "perl", "sys", "native"]
 end
 
-% For this function, Python is over 10x faster than Java
-
 o = stdlib.Backend(mfilename(), backend);
 b = o.backend;
 

example/BenchmarkSamepath.m

@@ -1,6 +1,13 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture("..")}) ...
     BenchmarkSamepath < matlab.perftest.TestCase
 
+properties
+exist = [".", pwd()]
+diff = [".", mfilename('fullpath') + ".m"]
+not = tempname()
+fun = @stdlib.samepath
+end
+
 properties(TestParameter)
 backend
 end
@@ -16,7 +23,7 @@
 
 function bench_exist(tc, backend)
 tc.startMeasuring()
-y = stdlib.samepath('.', pwd(), backend);
+y = tc.fun(tc.exist(1), tc.exist(2), backend);
 tc.stopMeasuring()
 
 tc.verifyEqual(y, true)
@@ -25,7 +32,7 @@ function bench_exist(tc, backend)
 
 function bench_diff(tc, backend)
 tc.startMeasuring()
-y = stdlib.samepath('.', mfilename('fullpath'), backend);
+y = tc.fun(tc.diff(1), tc.diff(2), backend);
 tc.stopMeasuring()
 
 tc.verifyEqual(y, false)
@@ -34,7 +41,7 @@ function bench_diff(tc, backend)
 
 function bench_not_exist(tc, backend)
 tc.startMeasuring()
-y = stdlib.samepath('not-exist', 'not-exist', backend);
+y = tc.fun(tc.not, tc.not, backend);
 tc.stopMeasuring()
 
 tc.verifyEqual(y, false)

example/BenchmarkSamepathRun.m

@@ -1,14 +1,16 @@
 function [r, s] =  BenchmarkSamepathRun()
+tname = "BenchmarkSamepath";
+
 %% Exist, same
-r.same = run_bench('BenchmarkSamepath/bench_exist');
+r.same = run_bench(tname + "/bench_exist");
 s.exist = sampleSummary(r.same);
 disp(sortrows(s.exist, "Median"))
 %% Exist, different
-r.diff = run_bench('BenchmarkSamepath/bench_diff');
+r.diff = run_bench(tname + "/bench_diff");
 s.diff = sampleSummary(r.diff);
 disp(sortrows(s.diff, "Median"))
 %% Not Exist
-r.not = run_bench('BenchmarkSamepath/bench_not_exist');
+r.not = run_bench(tname + "/bench_not_exist");
 s.not = sampleSummary(r.not);
 disp(sortrows(s.not, "Median"))
 
@@ -17,6 +19,6 @@
 
 function result = run_bench(name)
 suite = testsuite(name);
-exp = matlab.perftest.TimeExperiment.limitingSamplingError(MaxSamples=25, RelativeMarginOfError=0.1);
+exp = matlab.perftest.TimeExperiment.limitingSamplingError(MaxSamples=20, RelativeMarginOfError=0.1);
 result = exp.run(suite);
 end