Correct SUV calculation in get_suv_constants

[carlvillads]

Updates get_suv_constants in rhscripts/dcm.py.
SUV-bw should be calculated from the SeriesTime DICOM tag instead of the AcquisitionTime tag

• Add time_tag parameter to get_suv_constants, allowing to choose between using Series Time ((0008,0031)) or Acquisition Time ((0008,0032)) for time calculations, if Acquisition Time is still preferred.
• Updated dcmread with stop_before_pixels=True since pixel data is not needed for header information

[ChristianHinge]

Personligt vil jeg foretrække at funktionen giver en error hvis SeriesTime ikke eksisterer, frem for at defaulte til AcquisitionTime. Jeg frygter at det kan skabe nogle silent errors. SeriesTime bør altid eksistere for PET.

[carlvillads]

Personligt vil jeg foretrække at funktionen giver en error hvis SeriesTime ikke eksisterer, frem for at defaulte til AcquisitionTime. Jeg frygter at det kan skabe nogle silent errors. SeriesTime bør altid eksistere for PET.

Giver god mening – jeg får lige lavet en ny commit

[carlvillads]

@ChristianHinge hvad siger du til denne ændring?

...
# Scan time
if time_tag == 'series':
    acq_time = ds[0x0008,0x0031].value  # Series time
elif time_tag == 'acquisition':
    acq_time = ds.AcquisitionTime
else:
    raise ValueError("'time_tag' must be 'series' or 'acquisition'")
...

Har også dobbelttjekket selve tagget (for en sikkerheds skyld), og det burde være det korrekte

[carlvillads]

@ChristianHinge hvem skal merge?

[ChristianHinge]

lgtm :-)