Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Environment definition contains cucim, but we don't need/use it #749

Closed
1 task done
ant0nsc opened this issue Jun 21, 2022 · 0 comments · Fixed by #750
Closed
1 task done

Environment definition contains cucim, but we don't need/use it #749

ant0nsc opened this issue Jun 21, 2022 · 0 comments · Fixed by #750
Assignees
Labels
bug Something isn't working

Comments

@ant0nsc
Copy link
Contributor

ant0nsc commented Jun 21, 2022

Is there an existing issue for this?

  • I have searched the existing issues

Bug summary

The default Linux environment file contains the cucim package. To the best of my knowledge, this is not required for any of the InnerEye functionality. Cucim is only used in the hi-ml-histopathology module, which we don't really use in InnerEye.
This leads to problems as reported by @kh296 in #741

Code for reproduction

See https://github.com/microsoft/InnerEye-DeepLearning/issues/741

Actual outcome

n/a

Error messages

No response

Expected outcome

Creating a Conda environment on a non-GPU machine should work just fine.

System info

No response

AB#6272

@ant0nsc ant0nsc added the bug Something isn't working label Jun 21, 2022
@peterhessey peterhessey moved this to In Review in InnerEye-OSS Jun 21, 2022
Repository owner moved this from In Review to Done in InnerEye-OSS Jun 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants