testing-general#4
Merged
Merged
Conversation
Pinned specific versions in requirements.txt for stability and added Flask package. Updated Dockerfile to upgrade dependencies and ensure compatibility by explicitly installing blinker.
Delete shedule
…plots. Updated superpixel methods by including "FBM" and "CCL" timing metrics, modifying centroid plotting parameters, and standardizing axis labels to Spanish. Commented out unused visualization methods, adjusted plot sizes, and added `.png` output for all generated plots.
Replaced the image path in `main.py` to use a new directory (`images/c`) and reorganized image assets by renaming and adding images to the updated directory. Introduced a new plot (`times-plot.png`) to the project structure.
… update `figure.figsize` and font settings, and standardize axis labels. Comment out unused plotting functions.
…essing for PNG files, and calculate average timing metrics for all methods.
…and clean up dependencies - Implement batch processing to identify and exclude algorithm-named PNG files. - Replace wireframe plot with surface plot for better visualization. - Remove unused Flask dependency from `requirements.txt`.
…ganization and modularity - Encapsulate functionality into `ImageAs3D` class with `read_gray` and `convert_to_3d` methods.
Integrate threading support and update processing logic for dataset testings for future deprecation of the image_algorithm_performance_tester.py. Clean up unused assets and update comments for clarity in centroid_calculator.py.
Results saved by thread/directory in a CSV. Dictionary Keys were changed for headers concistency in the CSV files. Threads has been enabled.
… integrate local dataset handling, and update dependencies in `requirements.txt`.
…nused dataset import in `main.py`.
…szwalb) and refactor grayscale handling.
…es, centralize grayscale handling, add modular 3D wireframe plotting, and clean up redundant code.
Enhance wireframe plots with centroid markers, and fine-tune Quickshift parameters (stride and max_dist).
…ty and maintainability: - Replace single quotes with triple double quotes for consistency in docstrings. - Rename variables for uniform camel case style (e.g., `superpixels_SLIC_time` to `superpixels_slic_time`). - Fine-tune `Quickshift` parameters (`max_dist` reduced to 50) for faster plotting. - Optimize wireframe rendering (stride set to 1 for higher precision). - Remove unused imports and redundant variables. - Enhance script instructions and results visualization notes.
# Conflicts: # requirements.txt
alex-ogs
approved these changes
Jul 30, 2025
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
2 participants
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.
Updated readme