Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to add healthimaging as a dependency #21

Open
srdev-hardik opened this issue Oct 15, 2023 · 6 comments
Open

Unable to add healthimaging as a dependency #21

srdev-hardik opened this issue Oct 15, 2023 · 6 comments

Comments

@srdev-hardik
Copy link

Hi Team,

While run "yarn cli add-extension ohif-aws-healthimaging" to add healthimaging as a dependency, I am getting "Command failed: yarn add ohif-aws-healthimaging" error.

image

Please help me to resolve this issue.

Thanks.

@anuragp2018
Copy link

same issue

@HemantPatel30
Copy link

Can you try adding version, yarn cli add-extension ohif-aws-healthimaging 0.10.0

@HemantPatel30
Copy link

more apparently, you can debug CLI command through visual studio debug terminal & find actual root cause where it is failing.

Open debug terminal,
put break point in platform/cli/src/commands/addExtension.js
run cli command
you will be able to debug installNPMPackage by going into node_modules

@almvdma
Copy link

almvdma commented Sep 12, 2024

The NPM ohif-aws-healthimaging project README.md file refers to checking out v3-stable branch on the OHIF/Viewers repo. This seems to be out of date and will cause that error that @srdev-hardik experienced.

If, instead, you checkout the release/3.8 branch of the OHIF/Viewers repo as per the instructions in the master branch of this repo ohif-aws-imaging, this error does not occur (as of yarn 1.22.22 and yarn set version classic)

However, even following this repo's README causes an error no DICOMweb base url provided - calls that require a URL will fail (separate issue)

@bwilso65
Copy link

However, even following this repo's README causes an error no DICOMweb base url provided - calls that require a URL will fail (separate issue)

Yes!
I am stuck here too.

Do we have any resolution?

@bwilso65
Copy link

However, even following this repo's README causes an error no DICOMweb base url provided - calls that require a URL will fail (separate issue)

Yes! I am stuck here too.

Do we have any resolution?

I found the issue. In the repo's README, the URL they have for the viewer is:
http://localhost:3000/viewers?StudyInstanceUIDs=$DICOMStudyUIDHere&ImageSetID=$ImageSetIDHere

but it should be:
http://localhost:3000/viewer?StudyInstanceUIDs=$DICOMStudyUIDHere&ImageSetID=$ImageSetIDHere

note the /viewers (old) and /viewer (new) change.

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

No branches or pull requests

5 participants