James Butler

Results 385 comments of James Butler

@SachidanandAlle Does this sound reasonable to improve the security for MonaiLabel? This usage of dependabot uses automation to reduce the amount of human interaction required to stay on top of...

Based on https://packaging.python.org/en/latest/discussions/install-requires-vs-requirements/ it appears that requirements.txt and `install_requires` of setup.cfg serve different purposes. The setup.cfg should contain the minimal set of requirements (non hard-pinned) while requirements.txt can contain pinned...

Zarr 3.0 requires Python 3.11 and newer. This is because they follow [SPEC0](https://scientific-python.org/specs/spec-0000/) as seen in the PR that dropped Python 3.10 support in https://github.com/zarr-developers/zarr-python/commit/0e4cc7e6b1e11874197993b993d1a7834821f790. As previously mentioned at https://github.com/Project-MONAI/MONAI/issues/8303#issuecomment-2601244571,...

It appears you are suffering from an issue of pip cache. Referring to https://github.com/pytorch/vision?tab=readme-ov-file#installation there appears to be a tight coupling of which version of torchvision goes with a specific...

> This particular Ubuntu system provides Nvidia 535.138 driver version, which supports cuda 12.2. Cuda 12.2 is not listed in Pytorch's support matrix (it lists 11.8, 12.1, 12.4). So while...

@muratmaga On Windows with a cleared pip cache (%LOCALAPPDATA%\pip\cache) and a fresh Slicer 5.8.0 install, I installed PyTorch using the automatic computation backend and it appropriately pulled latest PyTorch 2.6.0+cu126...

It appears you got latest PyTorch (2.6.0) and latest Torchvision (0.21.0), however it appears to have retrieved PyTorch 2.6.0+cu124 which is actually the default on Linux, but then got Torchvision...

Reviewing https://download.pytorch.org/whl/torchvision/ there are 55 different whls when searching for 0.17.2. Including a macOS arm whl though of course no CUDA whl for macOS arm since they don’t have Nvidia...

> Therefore, it would be better to use "P" as default node name for point lists. Currently Markups Plane nodes use “P” as the default name so it would have...

I created this issue specifically for figuring out the method of doing the rename of vtkMRMLMarkupsFiducialNode to vtkMRMLMarkupsPointListNode and how that transition would go regarding deprecation or hard transition. Therefore...