Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make voxel_size a mandatory argument everywhere #976

Open
jgostick opened this issue Jul 10, 2024 · 0 comments
Open

Make voxel_size a mandatory argument everywhere #976

jgostick opened this issue Jul 10, 2024 · 0 comments
Labels
api PR tag for breaking changes easy

Comments

@jgostick
Copy link
Member

Assuming 1.0 is a recipe for trouble since this is 1 meter, which is a problem if people are expecting physical results.
@jgostick jgostick added easy api PR tag for breaking changes labels Jan 14, 2025
@jgostick jgostick moved this to Backlog in PoreSpy V3.0 Jan 15, 2025
@jgostick jgostick moved this from Backlog to API in PoreSpy V3.0 Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
api PR tag for breaking changes easy
Projects
PoreSpy V3.0
Status: API
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jgostick