Skip to content

Conversation

@owencjones
Copy link
Contributor

@owencjones owencjones commented Feb 6, 2023

Type adjustments and fixes to reduce number of static type analysis issues with linting before working on main ticket in affected area.

  • Introduction of several new types, to minimise use of Any types for unknown JSON structures, etc
  • Addition of missing type signatures to several functions, and function parameters.

@owencjones owencjones changed the title Io 665 housekeeping for io 622 Type fixes for several files. Feb 6, 2023
@owencjones owencjones changed the title Type fixes for several files. [IO-665][internal] Type fixes Feb 6, 2023
@linear
Copy link

linear bot commented Feb 6, 2023

IO-665 Necessary housekeeping to code affected

Some code will need to be changed only in a linting and type-checking way for this parent ticket, so I'm breaking this off to minimise pull request lint.

append: bool,
class_prompt: bool = True,
delete_for_empty: bool = False,
import_annotators: bool = False,
Copy link
Contributor

Choose a reason for hiding this comment

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

This is technically from other ticket

@owencjones owencjones merged commit c5743d2 into master Feb 7, 2023
@owencjones owencjones deleted the io-665_housekeeping_for_io-622 branch February 7, 2023 10:55
balysv added a commit that referenced this pull request Feb 24, 2023
…mports

* master:
  including release in dataset (#534)
  [IO-749][external] Added python support install on 3.7 (#540)
  Io 585 incorrect type annotation (#537)
  [IO-543][internal] Move Imports To Multiprocessing (#521)
  Version bump to 0.8.12
  HOTFIX: Version for unincluded file (#533)
  [IO-683][internal] Fix version parsing (#532)
  [SWAT-753][external]: Fix a 'darwin convert' to mask formats path issue (#531)
  Added nifti to supported formats. (#530)
  [IO-665][internal] Type fixes (#529)
  Allow passing a Client to get_dataset() to avoid authentication errors if environment variable isn't set or there is no config. (#522)
  Version bump to 0.8.11
  Version bump (#528)
  [SWAT-734][Internal] Fixing conversion to darwin_json_1.0 (#523)
  Version_bump_v0.8.9 (#527)
  Updating docs for release process (#525)
  [io-524][internal] Fix for windows builds
  [SWAT-716][internal]: Fix .json extension parsing for dataset splitting (#520)
  [IO-557][internal] Move to use Poetry for packaging and publishing (#512)
balysv added a commit that referenced this pull request Feb 24, 2023
…mports

* master:
  including release in dataset (#534)
  [IO-749][external] Added python support install on 3.7 (#540)
  Io 585 incorrect type annotation (#537)
  [IO-543][internal] Move Imports To Multiprocessing (#521)
  Version bump to 0.8.12
  HOTFIX: Version for unincluded file (#533)
  [IO-683][internal] Fix version parsing (#532)
  [SWAT-753][external]: Fix a 'darwin convert' to mask formats path issue (#531)
  Added nifti to supported formats. (#530)
  [IO-665][internal] Type fixes (#529)
  Allow passing a Client to get_dataset() to avoid authentication errors if environment variable isn't set or there is no config. (#522)
  Version bump to 0.8.11
  Version bump (#528)
  [SWAT-734][Internal] Fixing conversion to darwin_json_1.0 (#523)
  Version_bump_v0.8.9 (#527)
  Updating docs for release process (#525)
  [io-524][internal] Fix for windows builds
  [SWAT-716][internal]: Fix .json extension parsing for dataset splitting (#520)
  [IO-557][internal] Move to use Poetry for packaging and publishing (#512)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants