Upgrade LP -> 4.7.3, update ggsave.

alshan · alshan · commit cea80a226768 · 2025-09-12T16:20:29.000-04:00
diff --git a/demo/js-frontend-app/src/jsMain/resources/index.html b/demo/js-frontend-app/src/jsMain/resources/index.html
@@ -10,7 +10,7 @@
 </head>
 <body>
 <script type="text/javascript"
-        src="https://cdn.jsdelivr.net/gh/JetBrains/lets-plot@v4.7.2/js-package/distr/lets-plot.min.js"></script>
+        src="https://cdn.jsdelivr.net/gh/JetBrains/lets-plot@v4.7.3/js-package/distr/lets-plot.min.js"></script>
 <script src="js-frontend-app.js"></script>
 <div>
     <h2>Lets-Plot Kotlin/JS Demo.</h2>
diff --git a/gradle.properties b/gradle.properties
@@ -35,8 +35,8 @@ nexusPublish.version=1.3.0
 ksp.version=1.9.25-1.0.20
 jupyterApi.version=0.12.0-313
 
-# Also update JS version in <home>/demo/js-frontend-app/src/jsMain/resources/index.html
-letsPlot.version=4.7.2
+# Also update the JS version in <home>/demo/js-frontend-app/src/jsMain/resources/index.html
+letsPlot.version=4.7.3
 
 # https://geotoolsnews.blogspot.com/
 geotools.version=33.2
diff --git a/plot-api/src/jvmMain/kotlin/org/jetbrains/letsPlot/export/ggsave.kt b/plot-api/src/jvmMain/kotlin/org/jetbrains/letsPlot/export/ggsave.kt
@@ -20,17 +20,22 @@ private const val DEF_EXPORT_DIR = "lets-plot-images"
 
 /**
  * Exports plot to a file.
- * Supported formats: SVG, HTML, PNG, JPEG and TIFF.
+ * Supported formats: SVG, HTML, PNG, JPEG, and TIFF.
  * Note: in some configurations raster formats might not be supported.
  *
  * If `path` is not specified, the output file will be saved in
  * `${user.dir}/lets-plot-images`.
  *
  * ## Notes
  *
+ * Large plot dimensions without units require explicit unit specification.
+ * When `w` or `h` value exceeds 20 without specifying units (e.g., `ggsave(p, 300, 400)`),
+ * we ask to specify units explicitly:
+ * `ggsave(p, 300, 400, unit='px')` or `ggsave(p, 3, 4, unit='in')`.
+ *
  * The output format is inferred from the file extension.
  *
- * **For PNG and PDF:**
+ * **For PNG and PDF: **
  *
  * - If `w`, `h`, `unit`, and `dpi` are all specified:
  *
@@ -54,14 +59,14 @@ private const val DEF_EXPORT_DIR = "lets-plot-images"
  *         - The plot maintains its aspect ratio, preserving layout, tick labels, and other visual elements.
  *         - Useful for generating high-resolution images suitable for publication.
  *
- * **For SVG:**
+ * **For SVG: **
  *
  * - If `w`, `h`, and `unit` are specified:
  *
  *     - The plot's pixel size (default or set via [ggsize()][org.jetbrains.letsPlot.ggsize]) is ignored.
  *     - The output size is set from the given values.
  *
- * **For HTML:**
+ * **For HTML: **
  *
  * - If `w` and `h` are specified:
  *
@@ -192,22 +197,21 @@ private fun exportRasterImage(
     unit: PlotExportCommon.SizeUnit? = null,
     targetDPI: Number? = null
 ) {
-    // As of lets-plot version 4.7.2
-    // PlotImageExport calls SwingUtilities.invokeAndWait().
-    // Check if Swing is present.
+    // As of lets-plot version 4.7.3
+    // PlotImageExport uses java.awt.image.BufferedImage
+    // Check if java.awt is present.
     try {
-        Class.forName("javax.swing.SwingUtilities")
+        Class.forName("java.awt.image.BufferedImage")
     } catch (e: ClassNotFoundException) {
         throw IllegalStateException(
             """
             
-            Can't export plot to raster formats: Swing is not available in this environment.
-            Raster image export requires SwingUtilities which is not present in Android JRE or headless environments.
+            Can't export plot to raster formats: 'java.awt.image.BufferedImage' is not available in this environment.
+            Raster image export requires 'java.awt.image.BufferedImage' which is not present in Android JRE or headless environments.
         """.trimIndent()
         )
     }
 
-
     // lets-plot-image-export.jar might not be present in the classpath.
     val imageBytes: ByteArray = try {
 

-Original file line number
+Diff line change
 </head>
 <body>
 <script type="text/javascript"
 -        src="https://cdn.jsdelivr.net/gh/JetBrains/[email protected].2/js-package/distr/lets-plot.min.js"></script>
 +        src="https://cdn.jsdelivr.net/gh/JetBrains/[email protected].3/js-package/distr/lets-plot.min.js"></script>
 <script src="js-frontend-app.js"></script>
 <div>
     <h2>Lets-Plot Kotlin/JS Demo.</h2>
Original file line number	Diff line number	Diff line change
`@@ -20,17 +20,22 @@ private const val DEF_EXPORT_DIR = "lets-plot-images"`
`20`	`20`
`21`	`21`	`/**`
`22`	`22`	`* Exports plot to a file.`
`23`		`- * Supported formats: SVG, HTML, PNG, JPEG and TIFF.`
	`23`	`+ * Supported formats: SVG, HTML, PNG, JPEG, and TIFF.`
`24`	`24`	`* Note: in some configurations raster formats might not be supported.`
`25`	`25`	`*`
`26`	`26`	* If `path` is not specified, the output file will be saved in
`27`	`27`	* `${user.dir}/lets-plot-images`.
`28`	`28`	`*`
`29`	`29`	`* ## Notes`
`30`	`30`	`*`
	`31`	`+ * Large plot dimensions without units require explicit unit specification.`
	`32`	+ * When `w` or `h` value exceeds 20 without specifying units (e.g., `ggsave(p, 300, 400)`),
	`33`	`+ * we ask to specify units explicitly:`
	`34`	+ * `ggsave(p, 300, 400, unit='px')` or `ggsave(p, 3, 4, unit='in')`.
	`35`	`+ *`
`31`	`36`	`* The output format is inferred from the file extension.`
`32`	`37`	`*`
`33`		`- * For PNG and PDF:`
	`38`	`+ * For PNG and PDF: `
`34`	`39`	`*`
`35`	`40`	* - If `w`, `h`, `unit`, and `dpi` are all specified:
`36`	`41`	`*`
`@@ -54,14 +59,14 @@ private const val DEF_EXPORT_DIR = "lets-plot-images"`
`54`	`59`	`* - The plot maintains its aspect ratio, preserving layout, tick labels, and other visual elements.`
`55`	`60`	`* - Useful for generating high-resolution images suitable for publication.`
`56`	`61`	`*`
`57`		`- * For SVG:`
	`62`	`+ * For SVG: `
`58`	`63`	`*`
`59`	`64`	* - If `w`, `h`, and `unit` are specified:
`60`	`65`	`*`
`61`	`66`	`* - The plot's pixel size (default or set via [ggsize()][org.jetbrains.letsPlot.ggsize]) is ignored.`
`62`	`67`	`* - The output size is set from the given values.`
`63`	`68`	`*`
`64`		`- * For HTML:`
	`69`	`+ * For HTML: `
`65`	`70`	`*`
`66`	`71`	* - If `w` and `h` are specified:
`67`	`72`	`*`
`@@ -192,22 +197,21 @@ private fun exportRasterImage(`
`192`	`197`	`unit: PlotExportCommon.SizeUnit? = null,`
`193`	`198`	`targetDPI: Number? = null`
`194`	`199`	`) {`
`195`		`- // As of lets-plot version 4.7.2`
`196`		`- // PlotImageExport calls SwingUtilities.invokeAndWait().`
`197`		`- // Check if Swing is present.`
	`200`	`+ // As of lets-plot version 4.7.3`
	`201`	`+ // PlotImageExport uses java.awt.image.BufferedImage`
	`202`	`+ // Check if java.awt is present.`
`198`	`203`	`try {`
`199`		`- Class.forName("javax.swing.SwingUtilities")`
	`204`	`+ Class.forName("java.awt.image.BufferedImage")`
`200`	`205`	`} catch (e: ClassNotFoundException) {`
`201`	`206`	`throw IllegalStateException(`
`202`	`207`	`"""`
`203`	`208`
`204`		`- Can't export plot to raster formats: Swing is not available in this environment.`
`205`		`- Raster image export requires SwingUtilities which is not present in Android JRE or headless environments.`
	`209`	`+ Can't export plot to raster formats: 'java.awt.image.BufferedImage' is not available in this environment.`
	`210`	`+ Raster image export requires 'java.awt.image.BufferedImage' which is not present in Android JRE or headless environments.`
`206`	`211`	`""".trimIndent()`
`207`	`212`	`)`
`208`	`213`	`}`
`209`	`214`
`210`		`-`
`211`	`215`	`// lets-plot-image-export.jar might not be present in the classpath.`
`212`	`216`	`val imageBytes: ByteArray = try {`
`213`	`217`