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
OpenPNM wants 'pore.diameter' but snow returns 'pore.inscribed/equivalent/inscribed_diameter'. Same is true for several other properties ('throat.length'). I think it would be helpful to offer a collection of geometry models which maps the snow outputs to the openpnm inputs that openpnm expects. It should be done in such as way that you can then add a normal geometry model collection for computing other stuff like volumes and surface areas.
Actually, as I type this it occurs to me that it should work as follows:
mods1=op.models.collections.geometry.spheres_and_cylindersmods2=op.models.collections.geometry.snow_to_openpnm_mapping# or whatever name worksmods1.update(mods2)
pn.add_model_collection(mods1)
The second-last line is the key since it will overwrite any models that are present on spheres_and_cylinders that we don't want, like 'pore.diameter'.
The text was updated successfully, but these errors were encountered:
Hi, i found that module 'openpnm.models.collections.geometry' has no attribute 'snow_to_openpnm_mapping' for the current version(3.1.2). Would you mind sharing how to conduct model collection for importing snow networks? Thank you!
From Snow, we can get equivalent_diameter, then can we get pore.diameter and throat.diameter by this? Then add other needed properties for further simulations?
OpenPNM wants 'pore.diameter' but snow returns 'pore.inscribed/equivalent/inscribed_diameter'. Same is true for several other properties ('throat.length'). I think it would be helpful to offer a collection of geometry models which maps the snow outputs to the openpnm inputs that openpnm expects. It should be done in such as way that you can then add a normal geometry model collection for computing other stuff like volumes and surface areas.
Actually, as I type this it occurs to me that it should work as follows:
The second-last line is the key since it will overwrite any models that are present on
spheres_and_cylinders
that we don't want, like 'pore.diameter'.The text was updated successfully, but these errors were encountered: