Skip to content

Feature #1552 asl output change#1719

Merged
jan-petr merged 15 commits intodevelopfrom
feature-#1552-asl-output-change
Jan 6, 2025
Merged

Feature #1552 asl output change#1719
jan-petr merged 15 commits intodevelopfrom
feature-#1552-asl-output-change

Conversation

@maartenhammer
Copy link
Contributor

@maartenhammer maartenhammer commented Jun 14, 2024

closes #1552

@maartenhammer maartenhammer added bug Something isn't working feature New feature, enhancement or request aspire ASPIRE project related issues labels Jun 14, 2024
@maartenhammer maartenhammer added this to the Release 1.12.0 milestone Jun 14, 2024
@maartenhammer maartenhammer requested a review from jan-petr June 14, 2024 14:19
@jan-petr jan-petr linked an issue Jun 17, 2024 that may be closed by this pull request
Update PDF report for new x.output format #1552
Closed
14 tasks
@HenkMutsaerts HenkMutsaerts removed bug Something isn't working feature New feature, enhancement or request aspire ASPIRE project related issues labels Aug 13, 2024
@HenkMutsaerts HenkMutsaerts requested review from HenkMutsaerts and removed request for HenkMutsaerts January 2, 2025 09:31
maartenhammer and others added 15 commits January 6, 2025 19:00
@maartenhammer @jan-petr
#1552 Updated PDF report to accept new x.output'
9fdf28e
@maartenhammer @jan-petr
#1552 GenerateReport adjusted printing of text values to new x.output…
722b92b 
… format
@maartenhammer @jan-petr
#1552 GenerateReport now takes a bOverwrite value to rewrite the repo…
2bdbc0f 
…rt from default settings
@maartenhammer @jan-petr
#1552 GenerateReport updated print QC images to support the new x.out…
60055a3 
…put_IM format for ASL
@maartenhammer @jan-petr
#1552 Updated ExampleConfigReport in functions/QualityControl
762ea0d
@maartenhammer @jan-petr
#1552 Added number formatting for PDFReport values, default = %0.2f
151c054
@maartenhammer @jan-petr
#1552 GeneratePDF Fixed QC Images bug when missing
9baf091
@maartenhammer @jan-petr
#1552 xASL_qc_GeneratePdfConfig added a title before qc parameters to…
96b501f 
… default pdf
@HenkMutsaerts @jan-petr
#1552 bugfix: for 1:size() -> for 1:length()
2501bbc
@HenkMutsaerts @jan-petr
#1552 xASL_qc_GeneratePdfConfig: also allow non-structural non-asl mo…
93670e2 
…dules
@HenkMutsaerts @jan-petr
#1552 xASL_qc_GenerateReport: always overwrite
54b04c3
@HenkMutsaerts @jan-petr
#1552 xASL_qc_LoadPdfConfig: always overwrite
0375ed5
@HenkMutsaerts @jan-petr
#1552 xASL_qc_ParsePdfConfig: add filename feedback
35d3310 
printing per file takes long, the rest is pretty fast. So it helps to give feedback on the file we print
@HenkMutsaerts @jan-petr
#1552 xASL_qc_*PdfConfig: move config file to subject-folder
b335180
@HenkMutsaerts @jan-petr
#1552 xASL_qc_*: avoid deleting old PDF reports unless they have the …
f8d8a4e 
…same name
@jan-petr jan-petr force-pushed the feature-#1552-asl-output-change branch from eda18c4 to f8d8a4e Compare January 6, 2025 18:02
jan-petr
jan-petr approved these changes Jan 6, 2025
View reviewed changes
Copy link
Contributor

@jan-petr jan-petr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@jan-petr jan-petr merged commit f8d8a4e into develop Jan 6, 2025
@jan-petr jan-petr deleted the feature-#1552-asl-output-change branch January 6, 2025 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jan-petr jan-petr jan-petr approved these changes

Assignees

@jan-petr jan-petr

Labels

None yet

Projects

None yet

Milestone

Release 2.0.0

Development

Successfully merging this pull request may close these issues.

Update PDF report for new x.output format

3 participants

@maartenhammer @jan-petr @HenkMutsaerts