Skip to content

Conversation

@shaneahmed
Copy link
Member

  • Update links for katherdataset
  • Use small sample for kather test. Full zip is only for local tests.

- Update links for katherdataset
- Use small sample for kather test. Full zip is only for local tests.
- Fix bugs generated as a result relinking kather dataset
@shaneahmed shaneahmed assigned shaneahmed and unassigned shaneahmed Oct 27, 2021
@codecov
Copy link

codecov bot commented Oct 27, 2021

Codecov Report

Merging #173 (d33f9c4) into develop (926ee7f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #173   +/-   ##
========================================
  Coverage    99.79%   99.79%           
========================================
  Files           40       40           
  Lines         2959     2979   +20     
  Branches       492      499    +7     
========================================
+ Hits          2953     2973   +20     
  Misses           1        1           
  Partials         5        5           
Impacted Files Coverage Δ
tiatoolbox/models/dataset/info.py 100.00% <100.00%> (ø)
tiatoolbox/data/__init__.py 100.00% <0.00%> (ø)
tiatoolbox/cli/save_tiles.py 100.00% <0.00%> (ø)
tiatoolbox/cli/slide_info.py 100.00% <0.00%> (ø)
tiatoolbox/cli/stain_norm.py 100.00% <0.00%> (ø)
tiatoolbox/cli/read_bounds.py 100.00% <0.00%> (ø)
tiatoolbox/cli/tissue_mask.py 100.00% <0.00%> (ø)
tiatoolbox/tools/stainnorm.py 100.00% <0.00%> (ø)
tiatoolbox/tools/tissuemask.py 100.00% <0.00%> (ø)
tiatoolbox/cli/patch_predictor.py 100.00% <0.00%> (ø)
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 926ee7f...d33f9c4. Read the comment docs.

- Ignore coverage for local test
- Fix typo
- Add test for non-existing directory to improve coverage.
@shaneahmed shaneahmed added the refactoring Code Refactoring label Nov 1, 2021
"https://zenodo.org/record/53169/files/"
"Kather_texture_2016_image_tiles_5000.zip"
"https://tiatoolbox.dcs.warwick.ac.uk/datasets"
"/kather100k-validation-norm-subset-90.zip"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@John-P I think @simongraham is referring to this line.

"06_MUCOSA",
"07_ADIPOSE",
"08_EMPTY",
"BACK",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vqdang I think this is in line with kather100k set.

- Update links in tests
@shaneahmed shaneahmed merged commit c5c530e into develop Nov 4, 2021
@shaneahmed shaneahmed deleted the dev-optimise-testkatherdataset branch November 4, 2021 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Code Refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants