Posts by jweste

jweste · Aug 6th 2022

Hi Marcel,

I just did the QIDO/WADO version.

I will send to you after we solve the row/cols problem

jweste · Aug 6th 2022

Hi, Marcel

I am working in a QIDO version of the front/back end.

I have a problem with Columns and Rows parameters.

If these parameters are present in query the result is empty.

If I get rid of them the result is ok.

I need to pass this parameters due QIDO compliance.

Code

const test = {
NumberOfFrames: "",
BitsAllocated: "",
Columns: "",
Rows: "",
InstanceNumber: "",
RetriveURL: "",
TimezoneOffsetFromUTC: "",
InstanceAvailability: "",
SOPInstanceUID: "",
SOPClassUID: "",
QueryRetrieveLevel: "IMAGE",
StudyInstanceUID: "2.16.840.1.113669.632.20.121711.10000161048",
SeriesInstanceUID: "1.3.46.670589.11.17521.5.0.3124.2008081915302126599",
};
params = JSON.stringify(test);
....
const dicomweb = true; // indicate json format
const cmd = `--dolua:dofile([[${APIFOLDER}/queryfunctions.lua]]);images([[${CQAE}]],[[${params}]],[[${dicomweb}]])`;


function images(server,params, dicomweb)
local ae = server or servercommand('get_param:MyACRNema')
a=DicomObject:new(params)
b=remotequery(ae, 'IMAGE', a, dicomweb);
io.write(b or '')
end;


function remotequery(ae, level, q, dicomweb)
local isdicomweb=true
if (dicomweb == nil or dicomweb == '') then isdicomweb= false end;
local remotecode =
[[
local ae=']]..ae..[[';
local level=']]..level..[[';
local dicomweb=]]..tostring(isdicomweb)..[[;
local q=]]..q:Serialize()..[[;
local q2=DicomObject:new(); for k,v in pairs(q) do q2[k]=v end;
local r = dicomquery(ae, level, q2):Serialize(true,true,dicomweb);
local s=tempfile('txt') local f=io.open(s, "wb") f:write(r) returnfile=s f:close();
]]
local f = servercommand('lua:'..remotecode)
return f
end;

Display More

jweste · Aug 6th 2022

Hi, Marcel

Please, I believe the folder length problem was solved with the files I sent you.

Regards,

Luiz

jweste · Aug 5th 2022

Quote from marcelvanherk

Great work!

The long command line bug remains open. I mailed you about it.

Marcel

I think I found a native solution.

I am sending you a new file.

jweste · Aug 5th 2022

But, I am searching a native solution to node

jweste · Aug 5th 2022

Another solution is mklink:

https://docs.microsoft.com/pt-…n/windows-commands/mklink

jweste · Aug 5th 2022

A solution I have read is using:

Code

subst X: "C:\Folder1\Really Long Path\Such Recursion\So Deep\Wow"

jweste · Aug 5th 2022

Quote from marcelvanherk

Great work!

The long command line bug remains open. I mailed you about it.

Marcel

This is a problem with windows<10. But I will "google" about a solution.

You can see in https://docs.microsoft.com/en-…-limitation?tabs=registry

jweste · Aug 5th 2022

>>Everything seems to work well and it looks very pretty. However, scrolling through slices is very slow. And it only works on scroll (wheel) not the left mouse, which is a bit frustrating on my laptop.

When the image is multi-frame the scroll is very fast between frames.

But regarding the images from series, I was afraid that a great amount of memory would be used to load all images in a single shot.

Then, I choose load one by one when clicked or with scrolling with mouse wheel, as you ask before.

>>Also the slices are not in the right order. They should be sorted by InstanceNumber. Because of the direct mapping of dicom query results to json this can be easier done on the js side.

OK, I did

>>Since I am new to node.js, just as many conquest users will be. a small manual on how to configure, install and run the api and the app would be welcome, if we want to distribute it with conquest.

I agree. But, my english is not so perfect to make a manual without language errors.

But, I will do.

>>I quess eventually the app should only use qido and wado to conform to standards. WADO does provide an interface to compress the images, so it may not be necessary to add it in the DicomSlice interface. But of course yoy can easily add it if you want, you have full control of the code.

When I started to work on it, I hadn´t skills with qido,wado uri or wado-rs. (And I still don´t have much)

I wasn´t sure if will be possible get the conquest query results with node.

So, It´s the reason I started the tests with a owner api.

But, it will not be too hard change all to qido/wado.

I only need a little more time.

jweste · Aug 5th 2022

Quote from marcelvanherk

Conquest supports:

1.2.840.10008.1.2.5 RLE Lossless (YES decompress only)

1.2.840.10008.1.2.4.50 JPEG Baseline (Process 1 - 8 bit) YES

1.2.840.10008.1.2.4.51 JPEG Baseline (Processes 2 & 4 - 12 bit) YES

1.2.840.10008.1.2.4.57 JPEG Lossless, Nonhierarchical (Processes 14) YES (J2)

1.2.840.10008.1.2.4.70 JPEG Lossless, Nonhierarchical (Processes 14 [Selection 1]) YES (J1)

1.2.840.10008.1.2.4.80 JPEG-LS Lossless Image Compression YES, FAST AND GOOD COMPRESSION (JS)

1.2.840.10008.1.2.4.81 JPEG-LS Lossy (Near-Lossless) Image Compression YES, FAST AND GOOD COMPRESSION (J7)

1.2.840.10008.1.2.4.90 JPEG 2000 Image Compression (Lossless Only) YES, SLOW (JK)

1.2.840.10008.1.2.4.91 JPEG 2000 Image Compression YES, SLOW (JL)

1.2.840.10008.1.2.1.99 Deflate Transfer Syntax NO

Maybe, passing a param to lua function API to choice the compression will be a good add on.

What you think?

Every transfer syntax in Conquest has a shortcut? as J2, JK ....

jweste · Aug 5th 2022

Marcel,

Here is the transfersyntaxes supported by Cornerstone.

I am not a specialist about it. I am only curious.

https://github.com/cornerstone…/docs/TransferSyntaxes.md

jweste · Aug 5th 2022

Quote from marcelvanherk

Yeah,

compression makes it complicated. An unrelated question. Is there a key our mouse way to scroll through the slices in your cornerstone demo. Otherwise it works quite nicely.

Marcel

Hi, Marcel

I have solved the compress problem, and I put scroll slices when clicking on stack icon.

j2 is ok, n2 I didn´t have success with cornerstone

I am sending you a new zip pack

Regards, Luiz

jweste · Aug 5th 2022

Quote from marcelvanherk

You may also want to reduce the size of the thumbnails, 128 pixels would be just fine.

Marcel

No problem.

jweste · Aug 5th 2022

Quote from marcelvanherk

Hi,

two issues: the compression is not executed, x:Compress() returns a copy, the correct code therefore is:

x=x:Compress("j2")

I also fixed dgate to correctly pass even or odd length files (needs to be changed on client and server ends)

call as follows:

b=servercommand('lua:'..remotecode, "binary")

No need to add null bytes anymore.

dgate.zip

Marcel

Display More

Hi, Marcel

The fix is ok.

I found cornerstone needs to load plugins to work with compress dicom files.

I am trying to solve it using nuxtjs.

jweste · Aug 4th 2022

Hi, Marcel,

I just tested n2 and j2 and cornerstone works with both

jweste · Aug 4th 2022

Quote from marcelvanherk

Hi,

I could get your sample code to work, dgate triggered but did not connect to the server and the node code does not get any output. If I put the api on a shorter path, it does work!

Furthermore you use n2 compression. Can this be read by cornerstone. I would suggest j2.

Marcel

Display More

>> Can this be read by cornerstone. I would suggest j2.

Yes, It can. It´s working without problems in my tests.

But, we can test others cases if you have a image that can´t be read.

I just used 'n2', because I ask you about compression in a other post and you suggested 'n2'.

I´d appreciate if you send it to me.

Or you can just change it to j2 and test yourself.

Luiz

jweste · Aug 3rd 2022

Hi, Marcel

I have used windows in my dev PC with conquest.

In windows, I created a folder called "dgateAPI" with dgate.exe, dicom.ini, dgate.dic, lua5.1dll and my queryconsquest.lua file(containing the code to call conquest).

It´s enough and it´s working very well.

But I have a linux ubuntu server where I am going host my API.

My API, using the dgate needs to access a conquest remote server(other machine PC or Linux)

Then, what is the similar files I have to put in that folder using linux?

Is enough to use dgate with chmod 777 and dgate.dic?

Regards, Luiz

jweste · Aug 3rd 2022

Quote from marcelvanherk

Hi,

annotations are not implemented. Sorry about that.

Marcel

All right. Thanks

jweste · Aug 3rd 2022

Quote from marcelvanherk

Hi,

fixed a few bugs in the json parser (you are the first actual user!) and an issue serialising empty sequences.

Hope this works allright: dgate.zip

Marcel

Display More

Hi, Marcel

Thank you. It´s working.

Other question, please.

I´m playing with wadorequest as well. I´d like to know how annotation param works.

I tested with:

Code

localhost:3002/api/dicom/wadocgi?requestType=WADO&contentType=image/jpeg&seriesUID=1.3.46.670589.5.2.10.2156913941.892665339.860724&studyUID=1.3.46.670589.5.2.10.2156913941.892665384.993397&objectUID=1.3.46.670589.5.2.10.2156913941.892665339.718742&annotation=patient

Am the result is a jpeg image, but there is no annotations.

Code

function wadocgicommand(obj, lwfq, size, region, contentType, transferSyntax, anonymize, annotation, bridge)
  local command=string.format("wadorequest:%s,%s,%s,%s,%s,%s,%s,%s,%s", obj, lwfq, size, region, contentType, transferSyntax, anonymize, annotation, bridge)
  servercommand(command,'cgibinary')
end;
function wadocgi(tbl)
  loadstring("t = "..tbl)()   -- transform string to table t
  local obj=string.format("%s\\%s\\%s", t.studyUID, t.seriesUID, t.objectUID)
  local contentType=t.contentType or "image/jpeg"
  local rows=t.rows or "0"
  local columns=t.columns or "0"
  local size=string.format("%d/%d", tonumber(rows), tonumber(columns))
  local region = string.gsub(tregion or "", ',', '/')
  local windowCenter = math.floor(t.windowCenter+0.5)
  local windowWidth = math.floor(t.windowWidth+0.5)
  local frameNumber=t.frameNumber or "0"
  local imageQuality=t.imageQuality or "100"
  local lwfq=string.format("%d/%d/%d/%d", windowCenter, windowWidth, tonumber(frameNumber), tonumber(imageQuality))
  local anonymize=t.anonymize or ""
  local charset=t.charset or ""
  local presentationUID=t.presentationUID or ""
  local bridge=t.bridge or ""
  local transferSyntax=t.transferSyntax or ""
  local annotation = string.gsub(t.annotation or '', ',', '/')
  wadocgicommand(obj, lwfq, size, region, contentType, transferSyntax, anonymize, annotation, bridge)
end;

Display More

jweste · Aug 2nd 2022

Hi Marcel

I printed params before "a=DicomObject:new(params)" in the function images. The parameters are ok and passed.

Here is the params printed:

{"ConquestDevice":"","ConquestObjectFile":"","ConquestImageDate":"","ConquestImageTime":"","PatientID":"66971","PatientName":"","SeriesInstanceUID":"","InstanceNumber":"","ImageDate":"","ImageTime":"","StudyInstanceUID":"","SOPInstanceUID":"","NumberOfFrames":""}

the problem is in this line: "a=DicomObject:new(params)".

I think this line is taking away some json parameters as SOPInstanceUID.

function images(server,params)

local ae = server or servercommand('get_param:MyACRNema')

io.write(params) --test

a=DicomObject:new(params)

b=remotequery(ae, 'IMAGE', a);

io.write(b or '')

end;

Posts by jweste

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server

Web server