Feature/v2 enhanced analysis - #1
Merged
Merged
Conversation
- README: remove HuggingFace Spaces YAML frontmatter (was rendering as raw text on GitHub) - packages.txt: replace libgl1-mesa-glx (removed in Debian bookworm/trixie) with libgl1 - requirements.txt: use -e ".[gui]" to include matplotlib and seaborn for the Streamlit Cloud visualizations tab - streamlit_app.py: replace main() call (removed in new app.py) with exec() pattern so Streamlit's re-run model works correctly - test.yaml: drop poetry in favour of plain pip (avoids poetry.lock sync issues after adding opencv-python-headless); add libgl1/libglib2.0-0 apt step; run tests on feature/** branches too - conventional-prs.yaml: mark title check continue-on-error in fork (advisory only; lasisilab enforces it on the upstream PR) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…flict libglib2.0-0 from Bullseye repos requires libffi7 which is not available on the Trixie-based Streamlit Cloud image. opencv-python-headless needs no system GL libraries, so packages.txt can be empty. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…as package names apt-get was trying to install each word of the comment as a package.
Streamlit Cloud now runs Python 3.14.4. The previous caret constraints (^1.26.4 for numpy, ^0.22 for scikit-image, ^10.2 for pillow) forced pip to use source distributions that fail to compile on Python 3.14. Widening to >=X allows pip to resolve pre-built wheels for newer minor versions. Also remove quotes around .[gui] in requirements.txt for uv compatibility.
app.py was calling segment_section() with sam2_checkpoint=/sam2_cfg= (hair_analysis names) but section_sam2.py defines checkpoint=/model_cfg=.
- TestComputeEFD: access efd["efd_coeffs"] instead of treating return value as ndarray directly - TestComputeRadialProfile: update key names to radial_* prefix, replace missing radius_range_mu with computed radial_max - radial_min, add n_radial_peaks to expected key set - TestExtractFeaturesFromArray: fix call signature (resolution_mu=, source_name=), adapt assertions for dict return type, fix efd key count (51 not 40), fix hu prefix (no underscore) - TestClassifyShape: add n_radial_peaks and asymmetry_index to all feature dicts (now required by classify_shape) - test_section_pipeline: fix hu column prefix from hu_ to hu - TestCurlIndex: make _curved_skel dense enough to avoid disconnected segments causing nan; fix assertion direction (chord/arc metric gives lower values for curved lines, not higher) Agent-Logs-Url: https://github.com/ezhil384/fibermorph/sessions/86945c40-fef0-4344-97fa-c44ac3c5e80d Co-authored-by: ezhil384 <80913303+ezhil384@users.noreply.github.com>
…ctored-version fix(tests): align test suite with refactored v2 API
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
v2 updates