monodera / pfs_target_uploader

A web application to validate and upload target lists with pointing simulations for PFS observation
https://pfs-etc.naoj.hawaii.edu/uploader/
MIT License
1 stars 0 forks source link

fix plot's aspect ratio to fit all labels #247

Closed monodera closed 2 weeks ago

monodera commented 2 weeks ago

@wanqqq31 I found that the completion rate plots cannot show labels when more than ~5 priorities are in the target list. I made a quick fix (you can see the screenshot in the "File changed" section). Do you mind if this change is merged? I think you have some thoughts when you made the original plots.

wanqqq31 commented 2 weeks ago

Screenshot from 2024-07-02 12-52-06

Thanks for the revision. I personally do not like to extend the height of plots that much.. I made some revisions to separate the legend into two columns. Now I think all P0-9 label can be seen. But please let me know you have any concerns.

monodera commented 2 weeks ago

Thanks! I didn't know about these options for the legend configuration. I further made micro-adjustments.

The screenshot in the document is also updated.

monodera commented 2 weeks ago

Screenshot 2024-07-01 at 18 49 20 ppp_result_full

wanqqq31 commented 2 weeks ago

Thanks! It looks good.