Fix space in ds_qsiprep_5tt_hsvs

[tsalo]

Closes none, but addresses a bug identified by @araikes.

Changes proposed in this pull request

• Hardcode space entity in ds_qsiprep_5tt_hsvs as ACPC, not T1w.

[tsalo]

@mattcieslak should I also change the T1w default here?

qsirecon/qsirecon/workflows/recon/utils.py

Lines 54 to 60 in c1b8435

	def init_discard_repeated_samples_wf(
	inputs_dict,
	name="discard_repeats",
	qsirecon_suffix="",
	space="T1w",
	params={},
	):

The space parameter isn't actually used, so it might be better to just remove it.

[mattcieslak]

The space parameter isn't actually used, so it might be better to just remove it.

I think we should remove it

[mattcieslak]

Thanks!

[tsalo]

I'm pretty sure the hsvs test is going to fail, but I also noticed other cases where we have T1w.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 33.94%. Comparing base (8545323) to head (c280cb7).

Files with missing lines	Patch %	Lines
qsirecon/interfaces/scalar_mapping.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #175   +/-   ##
=======================================
  Coverage   33.94%   33.94%           
=======================================
  Files          57       57           
  Lines        7195     7195           
  Branches      983      983           
=======================================
  Hits         2442     2442           
  Misses       4655     4655           
  Partials       98       98           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests

[mattcieslak]

Those spaces in the csv file may be coming from here.