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.

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

Update grainstats entry point to work with ImageGrainCrops #1101

Open
ns-rse opened this issue Mar 4, 2025 · 0 comments
Open

Update grainstats entry point to work with ImageGrainCrops #1101

ns-rse opened this issue Mar 4, 2025 · 0 comments
Labels
GrainStats Issues pertaining to the GrainStats class

Comments

@ns-rse
Copy link
Collaborator

ns-rse commented Mar 4, 2025

#1022 restructures from a dictionary to add new classes for image grain crops (ImageGrainCrops).

Subject to...

a) TopoStats saving all of this raw data when saving to .topostats files.
b) updating AFMReader to reconstruct these when loading .topostats files.

...we will then have to update the grainstats entry point to use these.

Currently the tests/run_modules.py::test_grainstats() test is disabled.

The change(s) to make will be...

  1. Reconstructing the image_grain_crops (of class ImageGrainCrops and various attributes of different new classes).
  2. Passing this into the processing.run_grainstats() function when called from processing.process_grainstats().

For progress see also AFMReader #121.

@ns-rse ns-rse added the GrainStats Issues pertaining to the GrainStats class label Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GrainStats Issues pertaining to the GrainStats class
Projects
None yet
Development

No branches or pull requests

1 participant