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
I was using spheres_and_cylinders model and got negative conduit lengths. There was no error thrown until it was noticed that I had nans in my conductance. I had to trace that error all the way back to find a problem with conduit lengths. And in my exact case, the conduit lengths were negative because of overlapping pores where the pore center of the one pore was inside the other. I think this should be caught when conduit lengths are calculated. Just do a quick check for any negative conduit lengths and throw a warning at least.
The text was updated successfully, but these errors were encountered:
I was using
spheres_and_cylinders
model and got negative conduit lengths. There was no error thrown until it was noticed that I had nans in my conductance. I had to trace that error all the way back to find a problem with conduit lengths. And in my exact case, the conduit lengths were negative because of overlapping pores where the pore center of the one pore was inside the other. I think this should be caught when conduit lengths are calculated. Just do a quick check for any negative conduit lengths and throw a warning at least.The text was updated successfully, but these errors were encountered: