Page MenuHomePhabricator

OpenIGTLink support for the Ultrasound module
Closed, ResolvedPublic

Description

A branch which enables OpenIGTLink support for MITK-US was published on github:

https://github.com/Sereri/MITK/tree/Ultrasound-OpenIGTLink

This is a nice feature. Should we add it to the current master?

Event Timeline

User franza has pushed new remote branch:

bug-17944-OpenIGTLinkSupportMITK-US

User franza has pushed new remote branch:

bug-17944-OpenIGTLinkForUltrasoundIntegrationBranch

User franza has pushed new remote branch:

bug-17944-OpenIGTLinkUltrasoundIntegrationBranch2

Updating target milestone to upcoming release

A new OpenIGTLink module for MITK is planned and will be implemented soon. This will offer a more general way to use OpenIGTLink in MITK, also but not only for the Ultrasound module. We should wait for the new feature...

Regarding the old branches, they can serve as an example for the new implementation but I don't think it makes sense to merge them to the master.

This bug could not be fixed during the 2015.05 release.
Setting target_milestone to AfterNextRelease

User wagnerax has pushed new remote branch:

bug-17944-OpenIGTLinkPluginForUltrasound

User groehl has pushed new remote branch:

bug-17944-OpenIGTLinkPluginForUltrasoundJanekDebugMessages

User kirchnth has pushed new remote branch:

bug-17944-OpenIGTLinkPluginForUltrasoundTimestamped

User groehl has pushed new remote branch:

bug-17944-OpenIGTLinkUSIntegrationBranch

[4c1019]: Merge branch 'bug-17944-OpenIGTLinkUSIntegrationBranch'

Merged commits:

2015-10-07 16:18:51 Janek Gröhl [7070cd]
Merge branch 'master' of mitk.org:MITK into bug-17944-OpenIGTLinkUSIntegrationBranch


2015-10-07 14:05:57 Janek Gröhl [ba26d6]
Changed LAS to LPS

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-10-07 13:47:55 Janek Gröhl [abec3c]
Added a second mutex to counteract extremly slow transmission rates.

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-10-07 11:55:17 Janek Gröhl [e77b86]
Fixed spacing bug.

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-10-02 16:16:41 Janek Gröhl [fb8be9]
Formatting

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-10-02 15:42:28 Janek Gröhl [bdc0bc]
Fixed many bugs concerning multithreaded iterations of lists.

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-10-02 13:29:39 Janek Gröhl [ba176f]
Minor cleanups

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-10-02 13:19:51 Janek Gröhl [e56050]
Fixed a bug, where the project wuld not run in DEBUG Mode due to improper casting.
Extracted much duplicate code into a method.

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-10-01 19:47:51 Thomas Kirchner [fea4dc]
reformatting

Signed-off-by: Thomas Kirchner <t.kirchner@dkfz-heidelberg.de>


2015-10-01 19:39:59 Thomas Kirchner [04eddd]
always initialize a US stream with a random noisy image

Signed-off-by: Thomas Kirchner <t.kirchner@dkfz-heidelberg.de>


2015-09-30 17:30:54 Janek Gröhl [99aa1e]
Fixed a GUI labeling bug, where the context dependent text was not diyplayed correctly.

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-09-30 14:38:27 Janek Gröhl [804019]
Merge branch 'bug-17944-OpenIGTLinkPluginForUltrasound' into bug-17944-OpenIGTLinkUSIntegrationBranch

  1. Conflicts:
  2. Modules/OpenIGTLink/mitkIGTLMeasurements.cpp

2015-09-29 14:59:10 Thomas Kirchner [b0deff]
Merge branch 'bug-17944-OpenIGTLinkPluginForUltrasoundJanekDebugMessages' of mitk.org:MITK into bug-17944-OpenIGTLinkPluginForUltrasoundJanekDebugMessages

Conflicts:
Modules/OpenIGTLink/mitkIGTLClient.cpp
Modules/OpenIGTLink/mitkIGTLDevice.cpp


2015-09-29 14:55:26 Thomas Kirchner [598683]
remove unnecessary MITK_INFOs

Signed-off-by: Thomas Kirchner <t.kirchner@dkfz-heidelberg.de>


2015-09-28 17:15:30 Janek Gröhl [26a016]
Removed some debug messages.
More on wednesday.

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-09-28 16:54:29 Janek Gröhl [f0decd]
Added a parameter to the constructor and to the NewMacro of all IGTL Devices.
The parameter controls the "ReadFully - Flag" of the IGTL-Socket.

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-09-28 15:57:06 Thomas Kirchner [2ff519]
we want to read everything

Signed-off-by: Thomas Kirchner <t.kirchner@dkfz-heidelberg.de>


2015-09-25 13:07:51 Janek Gröhl [f9378e]
Added debug output for further investigation

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>


2015-09-22 15:42:41 Axel Wagner [19f8fd]
Add doc comments


2015-09-22 15:13:29 Axel Wagner [1ded57]
Both server and client need host and port


2015-09-22 14:08:18 Axel Wagner [f32296]
Make device name more descriptive


2015-09-22 12:54:23 Axel Wagner [dc33d5]
Fix build error


2015-09-22 12:32:01 Axel Wagner [528517]
Fix throwing


2015-09-22 11:32:50 Axel Wagner [2c1df5]
Remove USImageToIGTLMessageFilter


2015-09-18 18:37:01 Axel Wagner [4615f0]
Make UX nicer, add UI for server *and* client mode


2015-09-18 18:36:29 Axel Wagner [f4d9f7]
Also remove activation of plugin


2015-09-18 17:42:36 Axel Wagner [042891]
Remove OIGTL button


2015-09-18 17:39:03 Axel Wagner [c35327]
Have USDevices provide an OIGTL Server


2015-09-18 14:59:23 Axel Wagner [5fce1c]
Copy-paste code duplication


2015-09-18 13:21:01 Axel Wagner [55d621]
Remove spurious logging statement


2015-09-18 12:55:43 Axel Wagner [86c4e4]
Implement ImageToIGTLMessageFilter and plugin to test it


2015-09-17 16:20:18 Axel Wagner [dfb12d]
Get simple example of sending messages to work


2015-09-11 15:12:32 Axel Wagner [47d92f]
Just send something


2015-09-11 15:02:23 Axel Wagner [c0b5de]
Remove arpa/inet


2015-09-09 16:01:13 Axel Wagner [abcf3d]
First success reading ultrasound images from OpenIGTL

What works:

  • We can connect to an OIGTL server sending black and white images
  • We receive the images and show them on screen
  • We have basic GUI support for this

What doesn't work yet:

  • RAS coordinate systems
  • Color-images
  • Partial updates of the image
  • Taking into account the orientation of images

2015-09-09 16:00:38 Axel Wagner [f98acb]
Add breaks


2015-09-09 15:03:10 Axel Wagner [366df5]
Use itk byte swapping


2015-09-08 19:06:30 Axel Wagner [1eb427]
Correctly unwrap message


2015-09-08 14:56:53 Axel Wagner [08f85b]
Copy message, avoid race condition


2015-08-24 18:26:29 Axel Wagner [3e5d6e]
Merge remote-tracking branch 'origin/bug-19260-OpenIGTLinkMeasurementFeatures' into bug-17944-OpenIGTLinkPluginForUltrasound


2015-08-24 15:41:28 Axel Wagner [827978]
Merge remote-tracking branch 'remotes/origin/bug-19260-OpenIGTLinkMeasurementFeatures' into bug-17944-OpenIGTLinkPluginForUltrasound


2015-08-24 15:06:23 Axel Wagner [c457a4]
Pass port and hostname


2015-05-07 13:20:21 Axel Wagner [c7e245]
Implement IGTLMessage To USImage Filter


2015-08-24 14:41:20 Alfred Franz [f05ad0]
removed output


2015-08-24 14:38:59 Alfred Franz [1d3105]
activated measurement step 10 (rendering)


2015-08-24 13:42:34 Alfred Franz [7e404c]
added index to the time measurements


2015-08-24 11:36:09 Alfred Franz [061d4a]
added fps option to provider example


2015-08-24 11:24:29 Alfred Franz [1ab413]
switched to sequential player in provider example


2015-08-24 11:20:39 Alfred Franz [f23c07]
fixed the reader: works for more than one tool now


2015-08-19 17:17:04 Alfred Franz [75ba0a]
fixed waiting command: 10000 seconds was way to long, now waits 10 seconds for the reply.

User groehl has pushed new remote branch:

bug-17944-OpenIGTLinkTests

[f1d76e]: Merge branch 'bug-17944-OpenIGTLinkUSIntegrationBranchQuickFix'

Merged commits:

2015-10-07 17:31:31 Janek Gröhl [591c7b]
COMP Commented out defect test.

Signed-off-by: Janek Groehl <j.groehl@dkfz-heidelberg.de>

kirchnth added a subscriber: kirchnth.

This is done ... Has been done for a long timenow. Closing.