DnaChisel icon indicating copy to clipboard operation
DnaChisel copied to clipboard

Feature: PDF report generation optional during optimize_with_report

Open lifefoundry-scott opened this issue 5 months ago • 2 comments

Hey there,

Unfortunately we have found weasyprint to be a bit of a headache to deploy across different environments due to its external dependencies and differing installation across architectures. It would be very helpful if there were a flag to avoid pdf generation during this function while still exporting the other files (CSVs, genbanks).

Plateo requires pdf-reports (not as extra as in DNA Chisel, although I would appreciate that addition), so that makes it a bit hard to opt-out of this dependency chain.

Appreciate the work!

lifefoundry-scott avatar Jul 11 '25 04:07 lifefoundry-scott

Hi,

I'm open to this suggestion. It will require changing DnaOptimizationProblem.optimize_with_report() and the sister method in CircularDnaOptimizationProblem and write_optimization_report() called by these.

The two options are a flag parameter, or making the html/pdf generation dependent of PDF_REPORTS_AVAILABLE which is already in the code (I'm leaning towards this).

Also, if there's interest, I can enable saving the html version of the report as well.

veghp avatar Jul 11 '25 10:07 veghp

Both work for me! I would make use of html reports if they don't require the heavy dependencies for sure.

Much appreciated, thanks!

lifefoundry-scott avatar Jul 11 '25 22:07 lifefoundry-scott