Skip to content

DEV: Improve TIAToolbox Import#478

Merged
shaneahmed merged 19 commits intodevelopfrom
dev-improve-imports
Sep 27, 2022
Merged

DEV: Improve TIAToolbox Import#478
shaneahmed merged 19 commits intodevelopfrom
dev-improve-imports

Conversation

@shaneahmed
Copy link
Member

@shaneahmed shaneahmed commented Sep 24, 2022

  • Improve TIAToolbox import using lazy imports.
  • Improves importing import tiatoolbox by 10 times.
  • This will improve execution of cli.

- Improve tiatoolbox import using lazy import.

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Improve tiatoolbox import using lazy import.

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Sep 24, 2022

Codecov Report

Merging #478 (29b96f4) into develop (fafa65e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #478   +/-   ##
========================================
  Coverage    99.64%   99.64%           
========================================
  Files           60       61    +1     
  Lines         5926     5927    +1     
  Branches      1056     1056           
========================================
+ Hits          5905     5906    +1     
  Misses           9        9           
  Partials        12       12           
Impacted Files Coverage Δ
tiatoolbox/models/engine/__init__.py 100.00% <ø> (ø)
tiatoolbox/utils/__init__.py 100.00% <ø> (ø)
tiatoolbox/visualization/__init__.py 100.00% <ø> (ø)
tiatoolbox/data/__init__.py 100.00% <100.00%> (ø)
tiatoolbox/wsicore/__init__.py 100.00% <100.00%> (ø)
tiatoolbox/wsicore/metadata/__init__.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

- Fix path for imports

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for models

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for dsl

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for tiatoolbox.tools

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Run pip workflow only on relevant files

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for utils

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for wsicore

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for wsicore

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for registration

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for tileserver

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for tileserver

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
- Fix imports for models

Signed-off-by: Shan E Ahmed Raza <13048456+shaneahmed@users.noreply.github.com>
@shaneahmed shaneahmed requested a review from John-P September 24, 2022 17:13
@shaneahmed shaneahmed self-assigned this Sep 24, 2022
@shaneahmed shaneahmed added the enhancement New feature or request label Sep 24, 2022
@shaneahmed shaneahmed changed the title DEV: Improve TIAToolbox import DEV: Improve TIAToolbox Import Sep 25, 2022
Copy link
Contributor

@John-P John-P left a comment

Choose a reason for hiding this comment

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

Looks good, just a few minor comments.

- Run pip install workflow only on requirements file
…-only' into dev-improve-imports

# Conflicts:
#	.github/workflows/pip-install.yml
- Update `numpy` requirements
- Import importlib.util instead of util
@shaneahmed shaneahmed merged commit 2595d37 into develop Sep 27, 2022
@shaneahmed shaneahmed deleted the dev-improve-imports branch September 27, 2022 09:25
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.

2 participants