-
Notifications
You must be signed in to change notification settings - Fork 20
Temp fix for CRF #46
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
Merged
Merged
Temp fix for CRF #46
Conversation
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
Codecov Report
@@ Coverage Diff @@
## main #46 +/- ##
==========================================
+ Coverage 72.33% 82.06% +9.72%
==========================================
Files 50 43 -7
Lines 5209 4594 -615
==========================================
+ Hits 3768 3770 +2
+ Misses 1441 824 -617
... and 10 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
C-Achard
added a commit
that referenced
this pull request
Jul 28, 2023
C-Achard
added a commit
that referenced
this pull request
Jul 28, 2023
C-Achard
added a commit
that referenced
this pull request
Jul 28, 2023
Closed
C-Achard
added a commit
that referenced
this pull request
Sep 19, 2023
* Utils improvements - Set new results path now remains across utils - Utils generate one layer and update it on subsequent runs - Layer choice tries to be conserved across utils * Cleanup + CRF fixes * Fragmenting utility * Temp fix for CRF (#46) * Update plugin_convert.py * Update test_and_deploy.yml * Update test_inference.py * Fix tests + new weights * Update test_models.py * Fix dir for saving in tests * Docstring update * CRF and utils colormap adjustment * Update plugin_crf.py * Fix typo in crf.py Co-authored-by: Jessy Lauer <[email protected]> * Fix Pathlib typo in plugin_convert.py Co-authored-by: Jessy Lauer <[email protected]> * Fixed filepaths + comments cleanup * Fixed merge comment deletion --------- Co-authored-by: Jessy Lauer <[email protected]>
C-Achard
added a commit
that referenced
this pull request
Nov 10, 2023
* Add LayerNorm * Change softmax arg * Num group 2 * Update model.py * Update model.py * Reduce depth of WNet * Started WNet training UI * Workable WNet training prototype * Fixes * Test fixes * Temp fix for CRF (#46) * Minor fixes * Tests & training * Fix tests + new weights * Fix ETA precision * Docstring update * Update plugin_model_training.py * Update contrast limit when updating layers * Update config.py * Fixed normalization * Update plugin_model_training.py * Update workers_utils.py * Trying to fix input normalization * Fix name mismatch * Fix decoder evaluation * Update dice calculation * Update dice coeff * Update worker_training.py * Fix eval detach * Fix Dice list for WNet * Updated validation UI * Tooltips and show_results update * Plots update * Plot + log_parameters * Update worker_training.py * Disable WANDB for now + log param tweaks * UI/log tweaks * Functional WNet training * Clean exit / free memory attempt * Cleanup + tests - Removed previous train script - Fix tests - Enable test workflow on GH * Deploy memory usage fix in inference as well * Memory usage fix * UI tweak * WNet cleanup + supervised training improvements * Change Dice metric include_background for WNet To avoid Max Dice calculation * Set better default LR across un/supervised * Update model.py * Update WNet weights * Fix default LR + sup. test * Fix new unsup LR in tests * Fix dir for saving in tests * Testing fixes Due to Singleton Trainer widget * Test unsupervised training and raise coverage * WNet eval test * Fix order for model deletion * Extend supervised train tests * Started docs update * Update plugin_model_training.py * Fixed filepaths * Fix paths in test (use pathlib) * Updated workers config * Fixed parse_default_path test * Ignore wandb results in gitignore * Enable GH Actions tests on branch temporarily * Fixed deletion of Qt imports in interface * Reverted include_background=True in Dice * Reintroduced best Dice channel seeking + refacto * Improve filepath messages * Fix unsup image loading when not validating * Fix training tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
fix
Something needs to be or has been fixed
requirements
Related to installation requirements and optional dependencies
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.
Fixes issue #45