search

fact-project / aict-tools

Reproducible Artificial Intelligence for Cherenkov Telescopes
MIT License
6 stars 6 forks source link

x axis unit label in Bias and Resolution plot doesn't fit #159

Closed Hckjs closed 3 years ago

Hckjs commented 3 years ago

https://github.com/fact-project/aict-tools/blob/0298fb588800655c1aafd1a5707d63b2815248ac/aict_tools/plotting.py#L112-L115

Since the x-axis is log-scaled, the unit in the label should just be E_{MC}/TeV

maxnoe commented 3 years ago

@Hckjs yep, you're right. Could you open a PR with the fix?

maxnoe commented 3 years ago

And while you are at it, maybe change it to E_{{\mathrm{{true}}}?