@thomass This is the issue we talked about last week when you hat problem to use predicates to find some nodes by DICOM tags values.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Dec 14 2020
Dec 13 2020
- Could you provide also a SS example. As it should also work with SS and I would like to know the reason.
- A MAC version will be available with the next public MITK release (planned for next january).
- Thanks for the example data. I have some questions regarding it:
- a) acquesition 2592 containes 2 slices (with 21 time points/frequencies each). Is it meant to be one volume with 2 slices or are the 2 series but due to a multple slice sequence packed into one folder?
- b) what is the list.txt (see https://phabricator.mitk.org/w/mitk/cest/cest_user_manual/#list-txt-frequency-offsets-%CE%B4%CF%89) you used so far?
Dec 12 2020
Deleted branch bugfix/T28101-CMake3.19.
Pushed new branch bugfix/T28101-CMake3.19.
Will be fixed very soon in CMake 3.19.2: https://gitlab.kitware.com/cmake/cmake/-/issues/21529
Dec 11 2020
It works for me on the latest develop branch (#921705).
Dec 10 2020
I tried it with my current develop (not the latest due to ongoing QT issues - wip) and it resolves the last warning (predescibed dose) but not the others. According to the D449 this is the one you changed, so as expected I suppose :) As soon as I get the latest develop up and running I will check the diff further.
@thomass Please check the patch provided by https://phabricator.mitk.org/D449. It should solve you problem I hope. Would be cool if you could give feedback.
We need a dataset to reporduce the problem, to setup a testsetup.
Dec 9 2020
@kompan What is the status here? Is it finish. If not, what has to be done to close it. Should be done before we make a release in january and if much is left, we should know what to do. Thanks.
thanks for reply and your help!
- I solved the problem by changing from SS to multiple slice sequence. However, I am still struggling to get an CEST parameter map. Is there an updated manual available? I attached some sample data as well, see comment 3.
- Windows 10. But would prefer if there is a MAC version available.
- please find the sample data here: https://wetransfer.com/downloads/15f5ec7d7e9396f1e2d6736dee8d9eed20201209090335/c188ee6f22a20cefbc955818f0f5557c20201209090357/dc6f49
Its a mouse with brain tumor and I used the following parameters (7T MRI): CEST ampl. 4%, B1:3uT, delta t: 90us, sweep from 2000 Hz in 200Hz steps to -2000Hz (10 to -10 ppm).
I vaguely remember what it was used for: In the context of SemanticRelations but I can't find a related commit. So I guess we just leave it as it is and see if it pops up (once we talk again about the SemanticRelations and its future use).
Dec 8 2020
If this is now a crash but wasn't before, the TinyXML-2 migration is also a good candidate.
edit: Oh, was already authored in August. Sorry for the noise.
Might be related to T25305
@kompan Is this still open? You marked it in your report list as "in usage" or did you ment something else with "statsitical analysis of 17O data"?
@makr Could you please let us know:
- if the problem still persists
- What OS do you use?
- Could you provide any kind of example data for us to reproduce the error?
Dec 7 2020
Deleted branch bugfix/T28081-ShortcutsInWindowsInstallers.
Pushed new branch bugfix/T28081-ShortcutsInWindowsInstallers.
In T22412#215903, @kalali wrote:In T22412#93426, @goch wrote:This crutch should be removed when redesigning the tools.
@floca I stumbled upon this and I was wondering if this crutch can be removed with the changes made in T27490.
May be. We would need to test it.
@neher do we have suiteable testdata?
Deleted branch bugfix/T28080-RedistPackagingAndNSISGenerator.
In T22412#93426, @goch wrote:This crutch should be removed when redesigning the tools.
Pushed new branch bugfix/T28080-RedistPackagingAndNSISGenerator.
At least for XNAT this was recently resolved in T26449: XNAT connection failes due to missing OpenSSL connections.
would defer to after release
please close if ok
added to documentation of as.challenge():
(arg annotator:) If multiple annotators annotated the test cases, a string specifying the name of the column that contains the annotator identifiers. Only applies to rang-then-aggregate. Use with caution: Currently not tested.
added
podium(ranking)
to vignette.
podium() does have a different syntax for layouting (it is not ggplot2 but base graphics) but vignette does not describe layouting for ggplot2 plots either
report() does not know what type of consensus ranking method had been used.
I added sentence
"Consensus ranking according to mean ranks across tasks if method="euclidean" where in case of ties (equal ranks for multiple algorithms) the average rank is used, i.e. ties.method="average"."
to help for consensus() and would suggest to also add this to the vignette/readme, but not to report. Could someone of you please take care of this?
This task will be fixed with D447 (see https://phabricator.mitk.org/D447?vs=2005&id=2006#toc).
This task will be fixed with D447 (see D448#13472).