porespy icon indicating copy to clipboard operation
porespy copied to clipboard

Redundant `edges` argument in `pseudo_gravity_packing`

Open heinsimon opened this issue 11 months ago • 1 comments

The documentation and the function header include the argument 'edges'. Changing this argument from default value does not change anything in the output.

Checking the source code revealed: it is not used within the function itself.

I would expect either an error/warning when changing the argument from default value or not having this argument at all.

heinsimon avatar Mar 06 '24 12:03 heinsimon

That function is in need of some fixes. I think there are already some bug reports on the issue tracker so I'll affect this to the list.

jgostick avatar Mar 06 '24 13:03 jgostick