<html><head><style type='text/css'>p { margin: 0; }</style></head><body><div style='font-family: times new roman,new york,times,serif; font-size: 12pt; color: #000000'><br><br><hr id="zwchr"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><div dir="ltr">Hi Iulian, <div><br></div><div id="DWT1047">To differentiate in the test iGeom will have to have a version number associated with it that is accessible from Python (probably via another part of PyTAPS). Then you can compare the version number and execute the appropriate part of the test. </div></div></blockquote>yes, this is something I don't know how to do. Or what is the standard/preferred way to do it in python. <br><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><div dir="ltr"><div></div>
<div><br></div><div id="DWT1046">Arguably a better solution would be for the PyTAPS wrapper function to keep the same API it has now, test what version of igeom it has, and perform the appropriate call. This is a much better solution because then downstream code, tests included, won't need to change. </div></div></blockquote>this is a good idea, and I can do it without exporting anything else. It is just that the new argument (point) will always be [0,0,0],<br>so we cannot test anything else. If we always pass 0, why bother with modifying iGeom?<br><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><div dir="ltr"><div></div>
<div><br></div><div>For further arguments sake, it is probably not the right thing to do for the function signatures to change on micro version numbers. This is really bad for stability. I don't know what the specific use case is here,though, so maybe it is justified. </div>
<div id="DWT1048"><br></div></div></blockquote>yes, this is a good point. changing interface is never a good thing; and micro versions should never change function signatures. <br>It should be at least version 1.5.0 then :) <br><br>Thanks,<br>Iulian<br><br><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><div dir="ltr"><div></div><div>Be Well</div><div>Anthony</div></div><br></blockquote></div></body></html>