Automated commit of generated code

Author: github-actions[bot]

core/generated-sources/src/main/kotlin/org/jetbrains/kotlinx/dataframe/api/parse.kt

@@ -20,6 +20,8 @@ import java.time.format.DateTimeFormatter
 import java.util.Locale
 import kotlin.reflect.KProperty
 import kotlin.reflect.KType
+import kotlin.uuid.ExperimentalUuidApi
+import kotlin.uuid.Uuid
 
 /**
  * ### Global Parser Options
@@ -73,6 +75,16 @@ public interface GlobalParserOptions {
     public val nulls: Set<String>
 
     public val skipTypes: Set<KType>
+
+    /**
+     * Whether to allow parsing UUIDs to the experimental [kotlin.uuid.Uuid] type.
+     * By default, this is false and UUIDs are not recognized.
+     *
+     * NOTE: Interacting with a [Uuid][Uuid] in your code might require
+     * `@`[OptIn][OptIn]`(`[ExperimentalUuidApi][ExperimentalUuidApi]`::class)`.
+     * In notebooks, add `-opt-in=kotlin.uuid.ExperimentalUuidApi` to the compiler arguments.
+     */
+    public var parseExperimentalUuid: Boolean
 }
 
 /**
@@ -101,6 +113,11 @@ public interface GlobalParserOptions {
  * @param skipTypes a set of types that should be skipped during parsing. Parsing will be attempted for all other types.
  *   By default, it's an empty set. To skip all types except a specified one, use [convertTo] instead.
  * @param useFastDoubleParser whether to use [FastDoubleParser], defaults to `true`. Please report any issues you encounter.
+ * @param parseExperimentalUuid whether to allow parsing UUIDs to the experimental [kotlin.uuid.Uuid] type.
+ *   By default, this is false and UUIDs are not recognized.
+ *   NOTE: Interacting with a [Uuid][Uuid] in your code might require
+ *   `@`[OptIn][OptIn]`(`[ExperimentalUuidApi][ExperimentalUuidApi]`::class)`.
+ *   In notebooks, add `-opt-in=kotlin.uuid.ExperimentalUuidApi` to the compiler arguments.
  */
 public class ParserOptions(
     public val locale: Locale? = null,
@@ -110,8 +127,31 @@ public class ParserOptions(
     public val nullStrings: Set<String>? = null,
     public val skipTypes: Set<KType>? = null,
     public val useFastDoubleParser: Boolean? = null,
+    public val parseExperimentalUuid: Boolean? = null,
 ) {
 
+    /** For binary compatibility. */
+    @Deprecated(
+        message = PARSER_OPTIONS,
+        level = DeprecationLevel.HIDDEN,
+    )
+    public constructor(
+        locale: Locale? = null,
+        dateTimeFormatter: DateTimeFormatter? = null,
+        dateTimePattern: String? = null,
+        nullStrings: Set<String>? = null,
+        skipTypes: Set<KType>? = null,
+        useFastDoubleParser: Boolean? = null,
+    ) : this(
+        locale = locale,
+        dateTimeFormatter = dateTimeFormatter,
+        dateTimePattern = dateTimePattern,
+        nullStrings = nullStrings,
+        skipTypes = skipTypes,
+        useFastDoubleParser = useFastDoubleParser,
+        parseExperimentalUuid = null,
+    )
+
     /** For binary compatibility. */
     @Deprecated(
         message = PARSER_OPTIONS,
@@ -129,7 +169,31 @@ public class ParserOptions(
         nullStrings = nullStrings,
         skipTypes = null,
         useFastDoubleParser = null,
+        parseExperimentalUuid = null,
+    )
+
+    /** For binary compatibility. */
+    @Deprecated(
+        message = PARSER_OPTIONS_COPY,
+        level = DeprecationLevel.HIDDEN,
     )
+    public fun copy(
+        locale: Locale? = this.locale,
+        dateTimeFormatter: DateTimeFormatter? = this.dateTimeFormatter,
+        dateTimePattern: String? = this.dateTimePattern,
+        nullStrings: Set<String>? = this.nullStrings,
+        skipTypes: Set<KType>? = this.skipTypes,
+        useFastDoubleParser: Boolean? = this.useFastDoubleParser,
+    ): ParserOptions =
+        ParserOptions(
+            locale = locale,
+            dateTimeFormatter = dateTimeFormatter,
+            dateTimePattern = dateTimePattern,
+            nullStrings = nullStrings,
+            skipTypes = skipTypes,
+            useFastDoubleParser = useFastDoubleParser,
+            parseExperimentalUuid = null,
+        )
 
     /** For binary compatibility. */
     @Deprecated(
@@ -149,6 +213,7 @@ public class ParserOptions(
             nullStrings = nullStrings,
             skipTypes = skipTypes,
             useFastDoubleParser = useFastDoubleParser,
+            parseExperimentalUuid = null,
         )
 
     internal fun getDateTimeFormatter(): DateTimeFormatter? =
@@ -166,6 +231,7 @@ public class ParserOptions(
         nullStrings: Set<String>? = this.nullStrings,
         skipTypes: Set<KType>? = this.skipTypes,
         useFastDoubleParser: Boolean? = this.useFastDoubleParser,
+        parseExperimentalUuid: Boolean? = this.parseExperimentalUuid,
     ): ParserOptions =
         ParserOptions(
             locale = locale,
@@ -174,6 +240,7 @@ public class ParserOptions(
             nullStrings = nullStrings,
             skipTypes = skipTypes,
             useFastDoubleParser = useFastDoubleParser,
+            parseExperimentalUuid = parseExperimentalUuid,
         )
 
     override fun equals(other: Any?): Boolean {
@@ -188,6 +255,7 @@ public class ParserOptions(
         if (dateTimePattern != other.dateTimePattern) return false
         if (nullStrings != other.nullStrings) return false
         if (skipTypes != other.skipTypes) return false
+        if (parseExperimentalUuid != other.parseExperimentalUuid) return false
 
         return true
     }
@@ -199,11 +267,12 @@ public class ParserOptions(
         result = 31 * result + (dateTimePattern?.hashCode() ?: 0)
         result = 31 * result + (nullStrings?.hashCode() ?: 0)
         result = 31 * result + (skipTypes?.hashCode() ?: 0)
+        result = 31 * result + (parseExperimentalUuid?.hashCode() ?: 0)
         return result
     }
 
     override fun toString(): String =
-        "ParserOptions(locale=$locale, dateTimeFormatter=$dateTimeFormatter, dateTimePattern=$dateTimePattern, nullStrings=$nullStrings, skipTypes=$skipTypes, useFastDoubleParser=$useFastDoubleParser)"
+        "ParserOptions(locale=$locale, dateTimeFormatter=$dateTimeFormatter, dateTimePattern=$dateTimePattern, nullStrings=$nullStrings, skipTypes=$skipTypes, useFastDoubleParser=$useFastDoubleParser, parseExperimentalUuid=$parseExperimentalUuid)"
 }
 
 /** Tries to parse a column of strings into a column of a different type.

core/generated-sources/src/main/kotlin/org/jetbrains/kotlinx/dataframe/impl/api/parse.kt

@@ -142,6 +142,8 @@ internal object Parsers : GlobalParserOptions {
     override val skipTypes: Set<KType>
         get() = skipTypesSet
 
+    override var parseExperimentalUuid: Boolean = false
+
     override fun addDateTimePattern(pattern: String) {
         formatters.add(DateTimeFormatter.ofPattern(pattern))
     }
@@ -180,6 +182,7 @@ internal object Parsers : GlobalParserOptions {
             .let { formatters.add(it) }
 
         useFastDoubleParser = true
+        parseExperimentalUuid = false
         _locale = null
         nullStrings.addAll(listOf("null", "NULL", "NA", "N/A"))
     }
@@ -428,6 +431,8 @@ internal object Parsers : GlobalParserOptions {
         }
     }
 
+    private val uuidRegex = Regex("[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}")
+
     @OptIn(ExperimentalUuidApi::class)
     internal val parsersOrder = listOf(
         // Int
@@ -494,20 +499,25 @@ internal object Parsers : GlobalParserOptions {
         posixParserToDoubleWithOptions,
         // Boolean
         stringParser<Boolean> { it.toBooleanOrNull() },
-        // UUID
-        stringParser<Uuid> { str ->
-
-            val uuidRegex = Regex("[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}")
+        // Uuid
+        stringParserWithOptions<Uuid> { options ->
+            val parser = { str: String ->
+                val parseExperimentalUuid = options?.parseExperimentalUuid ?: this.parseExperimentalUuid
+                when {
+                    !parseExperimentalUuid -> null
+
+                    uuidRegex.matches(str) -> {
+                        try {
+                            Uuid.parse(str)
+                        } catch (_: IllegalArgumentException) {
+                            null
+                        }
+                    }
 
-            if (uuidRegex.matches(str)) {
-                try {
-                    Uuid.parse(str)
-                } catch (e: IllegalArgumentException) {
-                    null
+                    else -> null
                 }
-            } else {
-                null
             }
+            parser
         },
         // BigInteger
         stringParser<BigInteger> { it.toBigIntegerOrNull() },

core/generated-sources/src/test/kotlin/org/jetbrains/kotlinx/dataframe/api/parse.kt

@@ -489,7 +489,20 @@ class ParseTests {
     fun `parse valid Uuid`() {
         val validUUID = "550e8400-e29b-41d4-a716-446655440000"
         val column by columnOf(validUUID)
+        val parsed = column.parse(ParserOptions(parseExperimentalUuid = true))
+
+        parsed.type() shouldBe typeOf<Uuid>()
+        (parsed[0] as Uuid).toString() shouldBe validUUID // Change UUID to Uuid
+    }
+
+    @OptIn(ExperimentalUuidApi::class)
+    @Test
+    fun `parse valid Uuid with GlobalParserOptions`() {
+        val validUUID = "550e8400-e29b-41d4-a716-446655440000"
+        val column by columnOf(validUUID)
+        DataFrame.parser.parseExperimentalUuid = true
         val parsed = column.parse()
+        DataFrame.parser.resetToDefault()
 
         parsed.type() shouldBe typeOf<Uuid>()
         (parsed[0] as Uuid).toString() shouldBe validUUID // Change UUID to Uuid
@@ -500,6 +513,20 @@ class ParseTests {
     fun `parse invalid Uuid`() {
         val invalidUUID = "this is not a UUID"
         val column = columnOf(invalidUUID)
+        // tryParse as string is not formatted.
+        val parsed = column.tryParse(
+            ParserOptions(parseExperimentalUuid = true),
+        )
+
+        parsed.type() shouldNotBe typeOf<Uuid>()
+        parsed.type() shouldBe typeOf<String>()
+    }
+
+    @OptIn(ExperimentalUuidApi::class)
+    @Test
+    fun `do not parse Uuid by default`() {
+        val validUUID = "550e8400-e29b-41d4-a716-446655440000"
+        val column = columnOf(validUUID)
         val parsed = column.tryParse() // tryParse as string is not formatted.
 
         parsed.type() shouldNotBe typeOf<Uuid>()