porespy

porespy copied to clipboard

`pseudo_gravity` and `pseudo_electrostatic` packings should have `shape` or `im` as arguments

1

This is an api breaking change, so no rush.

jgostick

Ways to make a truly archived porespy installation?

7

My recent descent into dependency hell has me thinking about making truly 'archived' installations. I'm opening this issue to capture the ideas and options.

jgostick

Allow rescaling SNOW based on voxel size

1

Currently, voxel size is passed as an argument to SNOW, meaning that once the network is extracted, it's not possible to easily rescale the output based on a different voxel...

ma-sadeghi

Parallelize unit tests

2

Similar to PR #463, but for unit tests. Not urgent though, as our unit tests are still manageable.

ma-sadeghi

Use `fmm` to find perimeter of throats

1

We currently use the marching_cubes method to get more accurate surface area values. It got me thinking that throat perimeter could/should be done more accurately too. * Find the 3D...

jgostick

https://github.com/PMEAL/porespy/runs/4590324027?check_suite_focus=true

ma-sadeghi

Dask progress bars are not compatible with logging

1

I removed Dask progress bars for now (in PR #414). A more permanent solution would be to wait for Dask to fix the progress bar upstream. See dask/distributed#3414.

ma-sadeghi

The distance transform is useful in network extraction algs and therefore often needs to be computed anyway. Why not pass dt into _estimate_overlap instead of calculating it twice? I propose...

mkaguer

To reproduce: ```python import porespy as ps im = ps.generators.blobs([100, 100, 100], blobiness=0.75, divs=2) ```

ma-sadeghi

Currently, it shows the log generated by `pip install tensorflow`, etc. Should be turned off.

ma-sadeghi