assaytools icon indicating copy to clipboard operation
assaytools copied to clipboard

Improving xml2png.py

Open MehtapIsik opened this issue 9 years ago • 4 comments

I was testing xml2png.py with Infinite results of different assay types. I will list here some suggestions:

  1. Adding usage example with type argument will be useful here: https://github.com/choderalab/assaytools/blob/master/scripts/xml2png.py#L4
  2. For single_96 type, the plots are too wide. We should adjust plot dimensions. There is empty space on x-axis for wells bigger than 12 on 96-well plot. mi_flu_hsa_lig1_20150914_172954
  3. I think adding axis labels will be great. x-axis label: Well Index, y-axis label: Fluorescence
  4. Plot title usually has None printed for 'Part of Plate' section. image https://github.com/choderalab/assaytools/blob/master/scripts/xml2png.py#L111-L124 Do we need "Part of Plate" information? Or can we assign a default "Full Plate" when xml doesn't have this information.
  5. I don't have ant 384-well data, so I couldn't test singlet_384 type but I noticed it has hard coded ligand concentration(Lstated) for plotting. Can we remove that? https://github.com/choderalab/assaytools/blob/master/scripts/xml2png.py#L404-L418
  6. The difference between types spectra and scan wasn't obvious to me.

MehtapIsik avatar Aug 05 '16 19:08 MehtapIsik

Thanks Mehtap!

sonyahanson avatar Aug 10 '16 21:08 sonyahanson

Could it also be made python 3 compatible?

bas-rustenburg avatar Aug 24 '16 19:08 bas-rustenburg

It may also be helpful if it prints the help string if called without any arguments.

bas-rustenburg avatar Aug 24 '16 20:08 bas-rustenburg

All good ideas!

jchodera avatar Aug 24 '16 20:08 jchodera