Page MenuHomePhabricator

nolden (Marco Nolden)
UserAdministrator

Projects (11)

User Details

User Since
Aug 1 2016, 12:10 PM (398 w, 21 h)
Roles
Administrator

Recent Activity

Aug 2 2023

nolden added a comment to T30180: [Statistics] Median is approximated and completely off if sample size is small.

I had a look as well: two voxels is really an edge case, given the current method it would only give a correct result for number of bins == 1 . But setting the number of bins to at most (number of voxels - 1 ) doesn't really help, it fails e.g. for three voxels. In general thinking a bit about it I came to the conclusion that limiting the number of bins (currently it's _minimum_ 10) would still not yield a correct median.

Aug 2 2023, 5:09 PM · MITK (v2024.06)

Jul 12 2023

nolden closed T29622: How to Download Instagram Videos Easily and Quickly Using instafinstaHD as Invalid.
Jul 12 2023, 2:20 PM

Jun 22 2023

nolden added a comment to T27190: Clang format style for * and & at variables.

there is a DerivePointerAlignment option which basically unifies it on a file by file basis, in case there is no agreement. But I agree to Ralf with using the "left" style and put it next to the type

Jun 22 2023, 2:54 PM · Request for Discussion, MITK

Mar 19 2022

nolden committed rMITK51bd493c9c89: Fix clang warnings about unused variables (authored by nolden).
Fix clang warnings about unused variables
Mar 19 2022, 10:32 AM

Mar 17 2022

nolden added a comment to T28282: Workbench does not start when building with Qt 5.15 on Linux.

Just observed the same problem with latest "develop" of today.

Mar 17 2022, 5:24 PM · MITK

Oct 14 2021

nolden added a comment to T28428: Re-enable code coverage testing in MITK CI.

I found one last hint, about a possible interference with ccache , last try ...

Oct 14 2021, 3:47 PM · Auto-closed, CI, MITK
nolden added a comment to T28428: Re-enable code coverage testing in MITK CI.

I tried several things again, similar to described above, same situation.

Oct 14 2021, 3:38 PM · Auto-closed, CI, MITK

Oct 5 2021

nolden added members for Kaapana (internal): kochch, j162k.
Oct 5 2021, 11:27 AM

Jul 21 2021

nolden updated subscribers of T28529: Replace Elasticsearch and Kibana with apache-licensed opensearch.

Thanks Hanno. I think @schererj is the original author, so maybe he can comment or we discuss it in the meeting.

Jul 21 2021, 8:14 PM · Request for Discussion, Kaapana (internal)

Jul 1 2021

nolden added a comment to T28529: Replace Elasticsearch and Kibana with apache-licensed opensearch.

So this task as an _evaluation_ task has a high priority in my opinion. If it turns out to be a bigger thing to replace please stop and we should discuss again

Jul 1 2021, 1:47 PM · Request for Discussion, Kaapana (internal)
nolden added a comment to T28529: Replace Elasticsearch and Kibana with apache-licensed opensearch.

If this is not a big deal, i.e. if it really works after switching to the replacement there would be a big benefit. So it would be great if someone could at least try it and report some results, like how much work it would be. After that we could decide how to prioritize, but from my understanding it would make our Kaapana license task much easier

Jul 1 2021, 1:46 PM · Request for Discussion, Kaapana (internal)

Jun 28 2021

nolden added a member for MIC: forche.
Jun 28 2021, 9:49 AM

Jun 22 2021

nolden added a member for MIC: lucaskulla.
Jun 22 2021, 11:41 AM

Jun 21 2021

nolden added a member for MIC: nolden.
Jun 21 2021, 3:44 PM

Jun 8 2021

nolden moved T28529: Replace Elasticsearch and Kibana with apache-licensed opensearch from Backlog to Discussion on the Kaapana (internal) board.
Jun 8 2021, 11:09 AM · Request for Discussion, Kaapana (internal)
nolden triaged T28529: Replace Elasticsearch and Kibana with apache-licensed opensearch as Normal priority.
Jun 8 2021, 11:08 AM · Request for Discussion, Kaapana (internal)

Jun 3 2021

nolden updated subscribers of T26427: Support software rendering.

This seems to be still an issue. Maybe there is new information out there or it could be re-tested with newer Windows versions / drivers?

Jun 3 2021, 9:42 AM · Auto-closed, MITK

May 17 2021

nolden added a comment to T28232: Shapemodels.

Thanks. I think restricted access to code and models makes a big difference in the sense of preparatory work, code cleaning. model licensing etc.. ;) So that's why we thought it would be good first step to resolve all technical issues. Moving code and models to a public location would be another thing then.

May 17 2021, 11:28 AM · Auto-closed, Kaapana (internal), Request for Discussion, MITK

May 14 2021

nolden added a comment to T28232: Shapemodels.

Thanks @schererj , maybe one information was missing: as a first step we thought about making this build-able internally, so as part of the Kaapana/Dcipher-internal repo (option 2 in my comment above from March 25th)

May 14 2021, 12:41 PM · Auto-closed, Kaapana (internal), Request for Discussion, MITK

May 10 2021

nolden added a comment to T28232: Shapemodels.

@norajitr thanks for the follow up. Could you attach the two (?) Dockerfiles here and post the network drive path? Maybe someone else from Kaapana (internal) could have a look then, too.

May 10 2021, 5:20 PM · Auto-closed, Kaapana (internal), Request for Discussion, MITK

May 7 2021

nolden closed T28478: Update DCMTK version to 3.6.6 as Resolved.
May 7 2021, 9:30 AM · MITK (v2021.10)

May 6 2021

nolden committed rMITK121d8b57de63: Update DCMQI to official v1.2.4 (authored by nolden).
Update DCMQI to official v1.2.4
May 6 2021, 10:59 AM
nolden committed rMITK8a570d38b622: Update DCMQI to official v1.2.4 (authored by nolden).
Update DCMQI to official v1.2.4
May 6 2021, 10:59 AM
nolden committed rMITK6bffce5572f5: Update DCMTK to 3.6.6 (authored by nolden).
Update DCMTK to 3.6.6
May 6 2021, 10:58 AM
nolden committed rMITKeba7c6e68362: Update DCMTK to 3.6.6 (authored by nolden).
Update DCMTK to 3.6.6
May 6 2021, 10:58 AM

May 5 2021

nolden added a comment to T27437: Migrate MITK to use ITK 5.x .

Maybe something to keep an eye on: T28478 and https://github.com/InsightSoftwareConsortium/ITK/tree/master/Modules/ThirdParty/DCMTK

May 5 2021, 11:23 AM · MITK-ProjectTemplate, MITK (v2022.04)
nolden created T28478: Update DCMTK version to 3.6.6.
May 5 2021, 10:20 AM · MITK (v2021.10)

Apr 29 2021

nolden triaged T28470: "Dataset" option for ZIP upload as Wishlist priority.
Apr 29 2021, 3:15 PM · Kaapana (internal)

Apr 21 2021

nolden created T28455: RT struct 3D visibility is not working correctly.
Apr 21 2021, 11:30 AM · MITK (v2021.10)

Apr 14 2021

nolden added a comment to T27437: Migrate MITK to use ITK 5.x .

@kislinsk thanks for taking over. The WIP stuff I put on github still could be useful, see comment above. let me know if I can help.

Apr 14 2021, 9:57 AM · MITK-ProjectTemplate, MITK (v2022.04)

Apr 1 2021

nolden created T28428: Re-enable code coverage testing in MITK CI.
Apr 1 2021, 2:04 PM · Auto-closed, CI, MITK

Mar 25 2021

nolden added a comment to T28232: Shapemodels.

The topic came up in the Kaapana tech meeting again, because the current situation is causing problems there.

Mar 25 2021, 10:01 AM · Auto-closed, Kaapana (internal), Request for Discussion, MITK

Mar 15 2021

nolden added a comment to T27968: CTP gets stuck on large import.

Not sure if this is related:

Mar 15 2021, 1:47 PM · Kaapana (internal)
nolden removed a project from T28374: Workbench crashes after DICOM retrieval : Missing Info.
Mar 15 2021, 9:52 AM · Next Milestone, DICOM, MITK
nolden closed T28374: Workbench crashes after DICOM retrieval as Resolved.

@gaoh thanks for pointing there, I will close this one as a duplicate

Mar 15 2021, 9:51 AM · Next Milestone, DICOM, MITK
nolden added a comment to T28207: MITK DicomBrowser q/r Retrieve part crash.

Another option for testing is to use dicomserver.co.uk , I think it is preconfigured in the Workbench. It's a public DICOM endpoint and the data there differs, but 5 min ago I was able to reproduce the crash with a retrieval from there. Just activate the checkbox in the MITK config table , query last month, select a few patients and retrieve. Worked for me to reproduce the crash.

Mar 15 2021, 9:50 AM · MITK, Checklist

Mar 12 2021

nolden triaged T28374: Workbench crashes after DICOM retrieval as High priority.
Mar 12 2021, 3:45 PM · Next Milestone, DICOM, MITK

Feb 19 2021

nolden triaged T28323: Workbench aborts if passed an invalid command line option as Low priority.
Feb 19 2021, 5:50 PM · Auto-closed, MITK
nolden added a comment to T28282: Workbench does not start when building with Qt 5.15 on Linux.

Interesting catch. There is no call to regex_match in our code. If I understand it correctly the stack object is just created to check if an exception happens during construction, but maybe this is just another variant of the issue described for the library. A possible workaround could be to explicitly create and destroy the regex objects in our code, with explicit exception catching and re-throwing.

Feb 19 2021, 5:13 PM · MITK
nolden added a comment to T28232: Shapemodels.

@norajitr maybe a bit more background on the ticket for you:

Feb 19 2021, 10:39 AM · Auto-closed, Kaapana (internal), Request for Discussion, MITK

Feb 18 2021

nolden added a comment to T28302: MITK CI jobs for Kaapana.

I would like to have files versioned / with different names. If it's necessary to have a download link that doesn't change I can help with a script to create a "latest" link, had that for MITK nightlies back in the days

Feb 18 2021, 11:45 AM · CI, MITK

Feb 12 2021

nolden added a comment to T28302: MITK CI jobs for Kaapana.

Following this, are there some requirements that need to be specified here T28280?

Feb 12 2021, 12:02 PM · CI, MITK

Feb 10 2021

nolden added a comment to T28293: [MITK Workbench] Closing Workbench results in "Problem report" in macOS.

I tried running the Workbench on Linux both with valgrind as well as AdressSanitizer. Unfortunately there is a lot of noise from memory leaks and stuff, but I did not spot anything obvious which could be the cause.

Feb 10 2021, 10:47 AM · Auto-closed, MITK

Feb 9 2021

nolden added a comment to T28285: DICOM with 'ä' in filename not readable on Windows.

I did not dive into Windows APIs, so I just hope you know what you are doing ;)

Feb 9 2021, 12:27 PM · Checklist, MITK (v2022.10)
nolden added a comment to T28285: DICOM with 'ä' in filename not readable on Windows.

I did some Googling, it seems that starting with 2019 release, Windows 10 has a UTF8 mode. It's a bit difficult to enable: "old" control panel -> region -> Administrative settings -> Change System locale -> Enable UTF8

Feb 9 2021, 11:34 AM · Checklist, MITK (v2022.10)
nolden added a comment to T28285: DICOM with 'ä' in filename not readable on Windows.

That's a thorough dive in the topic ... Yes, toUtf8(), and nicely prints the correct Umlaut in the message box, telling you the file doesn't exist ;) I only had a quick glance around, and since a lot of stuff goes though ITK, and ITK decided to stay with local 8bit as far as I understand, I also think this would be the way to go. But I could really live with declaring this a "Known issue". Maybe something on the application user level would be nice, like

if toUtf8 != toLocal8Bit then show error
Feb 9 2021, 10:02 AM · Checklist, MITK (v2022.10)

Feb 8 2021

nolden added a comment to T28285: DICOM with 'ä' in filename not readable on Windows.

Ok, the GDCM unicode windows fix was introduced in the 3.0 series: https://github.com/malaterre/GDCM/commit/d4dfa144b941c6cc4da87f9795f7f5aba2bb7481

Feb 8 2021, 7:28 PM · Checklist, MITK (v2022.10)
nolden added a comment to T28285: DICOM with 'ä' in filename not readable on Windows.

Ok, I did look around a bit more. To summarize, I guess going for the "local 8bit representation" seems to be the most feasible way, since ITK also assumes this and I think we have a lot of ITK file handling code.

Feb 8 2021, 7:17 PM · Checklist, MITK (v2022.10)
nolden added a comment to T28285: DICOM with 'ä' in filename not readable on Windows.

Ok, I debugged to SetFileName @floca mentioned, it gets called from GDCMImageIO::InternalReadImageInformation in ITK

Feb 8 2021, 6:59 PM · Checklist, MITK (v2022.10)
nolden added a comment to T24402: Improve clang-format support.

Mostly as a reminder I tried on top of the branch above:

Feb 8 2021, 5:46 PM · MITK
nolden added a comment to T28285: DICOM with 'ä' in filename not readable on Windows.

Oh no, I can remember similar issues with German Ubuntu and "Arbeitsfläche" ... Basically, CI clients should probably always have a path compontent like /bäh / included to catch new issues like this one, and build system stuff, which is also a candidate.

Feb 8 2021, 3:00 PM · Checklist, MITK (v2022.10)

Feb 5 2021

nolden added a comment to T28283: Support building with Qt 5.15 .

I don't know why this was not linked automatically: https://phabricator.mitk.org/D469

Feb 5 2021, 3:50 PM · MITK (v2021.02)
nolden added a comment to T28283: Support building with Qt 5.15 .

I was hoping a bit to include it in the release, if the release would e.g. be used for kaapana containers for a longer time this could be helpful at some point. But of course it's not strictly necessary.

Feb 5 2021, 3:49 PM · MITK (v2021.02)
nolden added a comment to T28282: Workbench does not start when building with Qt 5.15 on Linux.

Valgrind shows some reading beyond memory boundaries, around mitkCoreActivator.cpp:129 and 218. Since it is the last line of all the AddPropertyPersistence calls there and all the creating of temp objects in there, I was wondering if it could be some reference to deleted temp objects issues. Just an idea.

Feb 5 2021, 3:31 PM · MITK
nolden added a comment to T28282: Workbench does not start when building with Qt 5.15 on Linux.

It's the CoreActivator if I read this correcty:

Feb 5 2021, 2:59 PM · MITK
nolden created T28283: Support building with Qt 5.15 .
Feb 5 2021, 2:14 PM · MITK (v2021.02)
nolden added a comment to T28282: Workbench does not start when building with Qt 5.15 on Linux.

And I just tested: as expected the unit tests do pass, I think it has to be a side effect of dlopen'ing the activator

Feb 5 2021, 2:13 PM · MITK
nolden updated subscribers of T28282: Workbench does not start when building with Qt 5.15 on Linux.

@floca the comment I mention was written by you, line 112, any spontaneous idea?

Feb 5 2021, 2:11 PM · MITK
nolden created T28282: Workbench does not start when building with Qt 5.15 on Linux.
Feb 5 2021, 2:05 PM · MITK
nolden added a comment to T28281: Review C++ warnings configuration.

In addition, suppressing deprecation warnings is in most cases no solution that lasts forever

Feb 5 2021, 2:00 PM · Auto-closed, Request for Discussion, MITK
nolden created T28281: Review C++ warnings configuration.
Feb 5 2021, 1:59 PM · Auto-closed, Request for Discussion, MITK
nolden created T28279: [Interaction] Improve image cropper cross hair interaction.
Feb 5 2021, 10:51 AM · Auto-closed, MITK
nolden triaged T28278: Image cropper doesn't work with PACS like mouse interaction as Low priority.
Feb 5 2021, 10:46 AM · Next Milestone, MITK
nolden triaged T28277: Empty Preferences page "Input Devices" as Low priority.
Feb 5 2021, 10:38 AM · MITK

Feb 1 2021

nolden added a comment to T28230: Issue with DICOM SEG -> NIFTI conversion: missing empty slices.

Who should I assign as reviewer for this?

Feb 1 2021, 5:31 PM · MITK

Jan 27 2021

nolden renamed T28206: [Image Statistics] Renaming segmentation or planar figure based ROI nodes breaks histogram from Renaming segmentation or planar figure based ROI nodes break histogram to Renaming segmentation or planar figure based ROI nodes breaks histogram.
Jan 27 2021, 10:50 AM · MITK (v2024.06)
nolden triaged T28206: [Image Statistics] Renaming segmentation or planar figure based ROI nodes breaks histogram as Unbreak Now! priority.
Jan 27 2021, 10:50 AM · MITK (v2024.06)
nolden added a comment to T27578: 2021.1 Linux installer does not work on Fedora.

Additional comment: the missing library doesn't prevent the Workbench from starting, but some/all DICOM functionality is missing which can be confusing

Jan 27 2021, 9:22 AM · MITK (v2022.10)
nolden raised the priority of T27578: 2021.1 Linux installer does not work on Fedora from High to Unbreak Now!.
Jan 27 2021, 9:18 AM · MITK (v2022.10)
nolden added a comment to T27578: 2021.1 Linux installer does not work on Fedora.

Ok, I just re-checked: Ubuntu 20.04 still has librwrap installed , so installer works. For Fedora there is no workaround , so either

Jan 27 2021, 9:18 AM · MITK (v2022.10)
nolden renamed T27578: 2021.1 Linux installer does not work on Fedora from Make sure libwrap isn't linked anymore to 2021.1 Linux installer does not work on Fedora.
Jan 27 2021, 9:18 AM · MITK (v2022.10)
nolden added a parent task for T27578: 2021.1 Linux installer does not work on Fedora: T28000: MITK v2021.02.
Jan 27 2021, 9:00 AM · MITK (v2022.10)
nolden added a subtask for T28000: MITK v2021.02: T27578: 2021.1 Linux installer does not work on Fedora.
Jan 27 2021, 9:00 AM · MITK-ProjectTemplate, MITK (v2021.02)

Jan 25 2021

nolden committed rMITKe02f5dfbc3b2: Summary: Applied some C++ style fixes (authored by nolden).
Summary: Applied some C++ style fixes
Jan 25 2021, 4:14 PM
nolden committed rMITK9fea04d2ed70: Applied some automated const FixIts to range-based loops (authored by nolden).
Applied some automated const FixIts to range-based loops
Jan 25 2021, 4:14 PM
nolden committed rMITK265001cf6a94: Added assignment operator to mitk::Vector (authored by nolden).
Added assignment operator to mitk::Vector
Jan 25 2021, 4:14 PM

Jan 21 2021

nolden renamed T28159: Officially support MITK using system Qt from Oficially support MITK using system Qt to Officially support MITK using system Qt.
Jan 21 2021, 9:41 AM · Auto-closed, CI, Request for Discussion, MITK
nolden added a comment to T28159: Officially support MITK using system Qt.

@kislinsk anyone from the MITK CI team who could take over?

Jan 21 2021, 9:38 AM · Auto-closed, CI, Request for Discussion, MITK

Jan 19 2021

nolden committed rMITK15ecdf8d92aa: Update DCMQI to v1.2.3 (authored by nolden).
Update DCMQI to v1.2.3
Jan 19 2021, 2:54 PM
nolden committed rMITKad45666d2a3f: Update DCMQI to v1.2.3 (authored by nolden).
Update DCMQI to v1.2.3
Jan 19 2021, 2:54 PM
nolden triaged T28174: Fix minor C++ style issues as Normal priority.
Jan 19 2021, 10:05 AM · MITK

Jan 15 2021

nolden added a comment to T28159: Officially support MITK using system Qt.

Thank. New command-line interface looks nice, but the challenge is where to put the credentials, especially if you want to put the script open source like in the Kaapana dockerfiles.

Jan 15 2021, 12:14 PM · Auto-closed, CI, Request for Discussion, MITK

Jan 14 2021

nolden added a project to T28159: Officially support MITK using system Qt: CI.
Jan 14 2021, 3:51 PM · Auto-closed, CI, Request for Discussion, MITK
nolden updated the task description for T28159: Officially support MITK using system Qt.
Jan 14 2021, 3:50 PM · Auto-closed, CI, Request for Discussion, MITK
nolden created T28159: Officially support MITK using system Qt.
Jan 14 2021, 3:46 PM · Auto-closed, CI, Request for Discussion, MITK

Dec 2 2020

nolden added a comment to T28049: DataUpload fails with Firefox.

PS: Chrome upload works but data doesn't show up in meta

Dec 2 2020, 1:07 PM · Kaapana (internal)
nolden created T28049: DataUpload fails with Firefox.
Dec 2 2020, 1:00 PM · Kaapana (internal)

Nov 27 2020

nolden committed rMITK6eaeaca7975f: Fix uninitialized reference warning/error (authored by nolden).
Fix uninitialized reference warning/error
Nov 27 2020, 9:21 AM
nolden committed rMITK857976ed83c8: Fix uninitialized reference warning/error (authored by nolden).
Fix uninitialized reference warning/error
Nov 27 2020, 9:20 AM

Nov 26 2020

nolden added a comment to T28006: MatchPointRegistration module compilation fails with Clang 11.

https://phabricator.mitk.org/D443

Nov 26 2020, 4:20 PM · MITK
nolden added a comment to T28006: MatchPointRegistration module compilation fails with Clang 11.

I can confirm this fixes the issue, I can merge it tomorrow

Nov 26 2020, 4:18 PM · MITK

Nov 24 2020

nolden created T28006: MatchPointRegistration module compilation fails with Clang 11.
Nov 24 2020, 9:25 AM · MITK

Nov 23 2020

nolden committed rMITKff0027eee2cb: Patch VTK to fix Freetype build error on GCC 10 (authored by nolden).
Patch VTK to fix Freetype build error on GCC 10
Nov 23 2020, 5:39 PM
nolden committed rMITK3ee32df042fb: Patch VTK to fix Freetype build error on GCC 10 (authored by nolden).
Patch VTK to fix Freetype build error on GCC 10
Nov 23 2020, 5:38 PM
nolden triaged T27995: Upgrade CDash to 3.0 as Wishlist priority.
Nov 23 2020, 2:04 PM · CI, MITK
nolden added a comment to T27959: MITK with VTK 9 fails on Fedora CI clients.

I can confirm this patches fixes the problem. @kislinsk : hotfix oder feature branch?

Nov 23 2020, 1:49 PM · MITK
nolden added a comment to T27959: MITK with VTK 9 fails on Fedora CI clients.
diff --git a/Rendering/FreeType/vtkFreeTypeTools.cxx b/Rendering/FreeType/vtkFreeTypeTools.cxx
index c54289dc60..03b899c4da 100644
--- a/Rendering/FreeType/vtkFreeTypeTools.cxx
+++ b/Rendering/FreeType/vtkFreeTypeTools.cxx
@@ -378,8 +378,7 @@ FTC_CMapCache* vtkFreeTypeTools::GetCMapCache()
 }
Nov 23 2020, 1:34 PM · MITK

Nov 17 2020

nolden created T27968: CTP gets stuck on large import.
Nov 17 2020, 3:50 PM · Kaapana (internal)

Oct 20 2020

nolden added a comment to T27902: Handle slashes in installer names.

Ups ;) Replace with - or _?

Oct 20 2020, 2:10 PM · MITK (v2021.02)

Oct 19 2020

nolden added a comment to T27894: Linux nightly installer doesn't start on Fedora.

I was just about to add a note that there is the general question how to handle python in installers. I guess for just having the basic runtime available it would be ok to just load libpython3.so or libpython3.8.so and let the system take the right one from the right path. But on Windows I guess most people don't install Python in system locations. Anyway I think this could be fixed even with the current packaging if we need it for 2020, but I'm just reporting, no pressure from my side ;)

Oct 19 2020, 1:25 PM · MITK (v2021.02)