Refactor: commented out ViT-related mentions in files #223
Conversation
This file contains 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
|
Nice PR, thanks! Two more things:
E File "/home/runner/work/aucmedi/aucmedi/tests/test_architectures_image.py", line 749
E def test_ConvNeXtBase(self):
E IndentationError: unexpected indentBR DM |
muellerdo
reviewed
Jul 16, 2024
README.md
Outdated
| @@ -14,7 +14,7 @@ The open-source software AUCMEDI allows fast setup of medical image classificati | |||
| - Wide range of 2D/3D data entry options with interfaces to the most common medical image formats such as DICOM, MetaImage, NifTI, PNG or TIF already supplied. | |||
| - Selection of pre-processing methods for preparing images, such as augmentation processes, color conversions, windowing, filtering, resizing and normalization. | |||
| - Use of deep neural networks for binary, multi-class as well as multi-label classification and efficient methods against class imbalances using modern loss functions such as focal loss. | |||
| - Library from modern architectures, like ResNet up to EfficientNet and Vision-Transformers (ViT). | |||
| - Library from modern architectures, like ResNet up to EfficientNet. <!-- and Vision-Transformers (ViT).--> | |||
- commented out ViT-related unittests to avoid IndentationError - Updated README (from Architecture EfficientNet to Architecture ConvNeXt) - Removed tensorflow-addons from requirements.txt and setup.py - Changed import of F1Score in file aucmedi/automl/block_train.py
muellerdo
reviewed
Jul 16, 2024
requirements.txt
Outdated
There was a problem hiding this comment.
Here, the commented vit-keras is missing, but is fine and could be added later if vit-keras gets hopefully an update in the future.
muellerdo
reviewed
Jul 16, 2024
added 2 commits
July 17, 2024 12:26
…quirements.txt - Removed num_classes parameter from F1Score in the AutoML training block to align with updated tf.keras - Added and commented out vit-keras in requirements.txt
|
Nice! Requirements are running through :) |
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.
This pull request addresses issue #220.
Changes made:
vit-kerasdependency fromrequirements.txt.Files modified:
README.mdaucmedi/neural_network/architectures/image/__init__.pyrequirements.txtsetup.pytests/test_architectures_image.pyPlease review the changes and let me know if there are any questions or further adjustments needed.