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

Improving Stage 1 Validity and Adding Utility Scripts #10

Open
wants to merge 32 commits into
base: master
Choose a base branch
from

Conversation

jchen42703
Copy link
Owner

@jchen42703 jchen42703 commented Feb 10, 2020

  • Fixed the way the dataset is split and sampled to prevent data leakage
  • TestVoxelDataset now has the argument load_labels
  • Added compatibility for inference on the preprocessed sliced 2D dataset
  • Added the script, kits19-2d-reproduce\scripts\utility\get_validation_predictions.py, for viewing validation predictions (to check potential disparities between validation and test performance)

@jchen42703 jchen42703 added the enhancement New feature or request label Feb 10, 2020
@jchen42703 jchen42703 self-assigned this Feb 10, 2020
PredictionDataset is for generating arrays for evaluation and ensembling
…and kidney tumor dices

Doing so to check why my tk and tu dice for training are so low while training results report good tk and tu dice scores
since the albumentations version doesn't transpose mask
…e-fly sampling support again

doing so bc gianmarco said that on the fly sampling is bueno and I think so too
…nTheFly and alphabetically sorted the imports
…y arg in io_params

When `sample_on_the_fly=True`, the json loaded must be the `slice_indices_json_path` json
…to fetch the class to cleanly support the OnTheFly datasetss

doing so to reduce repeat code in a readable way
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant