search

jrkerns / pylinac

An image analysis library for medical physics
https://pylinac.readthedocs.io/en/latest/
MIT License
153 stars 98 forks source link

PicketFence: Text superposition in publish_pdf #453

Closed LuisOlivaresJ closed 1 year ago

LuisOlivaresJ commented 1 year ago

I am using pylinac 3.11.0

Describe the bug

"Metadata" and "results" texts superposition in PicketFence.publish_pdf report. This bug has been previously described by Chase Mallory

To Reproduce

    from pylinac import PicketFence

    pf_img = 'RI.dcm'
    pf = PicketFence(pf_img)
    pf.analyze(tolerance = 0.4, action_tolerance = 0.2)

    pf.publish_pdf('PF_Stat_0_test.pdf', metadata = {'Author': 'Luis', 'unit': 'Clinac-iX'})

Screenshots

Captura

Proposed solution From https://github.com/jrkerns/pylinac/blob/5894520e90d45d74a1a1d184eeed0121a2238afd/pylinac/picketfence.py#L981-L982 Change results location, it could be:

location = (1.5, 22)

I'll try to make a pull request for this.