Automated commit of generated code

Author: github-actions[bot]

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

@@ -8,6 +8,7 @@ import org.jetbrains.kotlinx.dataframe.DataColumn
 import org.jetbrains.kotlinx.dataframe.DataFrame
 import org.jetbrains.kotlinx.dataframe.DataRow
 import org.jetbrains.kotlinx.dataframe.aggregation.AggregateGroupedBody
+import org.jetbrains.kotlinx.dataframe.annotations.AccessApiOverload
 import org.jetbrains.kotlinx.dataframe.api.asColumnGroup
 import org.jetbrains.kotlinx.dataframe.api.asDataColumn
 import org.jetbrains.kotlinx.dataframe.api.cast
@@ -25,6 +26,7 @@ import org.jetbrains.kotlinx.dataframe.impl.aggregation.AggregatableInternal
 import org.jetbrains.kotlinx.dataframe.impl.aggregation.GroupByReceiverImpl
 import org.jetbrains.kotlinx.dataframe.impl.columns.resolveSingle
 import org.jetbrains.kotlinx.dataframe.io.renderToString
+import org.jetbrains.kotlinx.dataframe.util.DEPRECATED_ACCESS_API
 import kotlin.reflect.KProperty
 
 internal const val UNNAMED_COLUMN_PREFIX = "untitled"
@@ -126,9 +128,13 @@ internal open class DataFrameImpl<T>(cols: List<AnyCol>, val nrow: Int) :
             selector = column,
         ).singleOrNull()
 
+    @Deprecated(DEPRECATED_ACCESS_API)
+    @AccessApiOverload
     override fun <R> getColumnOrNull(column: ColumnReference<R>): DataColumn<R>? =
         column.resolveSingle(this, UnresolvedColumnsPolicy.Skip)?.data
 
+    @Deprecated(DEPRECATED_ACCESS_API)
+    @AccessApiOverload
     override fun <R> getColumnOrNull(column: KProperty<R>): DataColumn<R>? = getColumnOrNull(column.toColumnAccessor())
 
     override fun getColumnOrNull(path: ColumnPath): AnyCol? =

core/generated-sources/src/main/kotlin/org/jetbrains/kotlinx/dataframe/impl/columns/DataColumnImpl.kt

@@ -30,20 +30,20 @@ internal abstract class DataColumnImpl<T>(
 
             this.isPrimitiveArray ->
                 type.isPrimitiveArray &&
-                    this!!::class.qualifiedName == type.classifier?.let { (it as KClass<*>).qualifiedName }
+                    this::class.qualifiedName == type.classifier?.let { (it as KClass<*>).qualifiedName }
 
             this.isArray -> type.isArray
 
             // cannot check the precise type of array
-            else -> this!!::class.isSubclassOf(type.classifier as KClass<*>)
+            else -> this::class.isSubclassOf(type.classifier as KClass<*>)
         }
 
     init {
         // Check for [Issue #713](https://github.com/Kotlin/dataframe/issues/713).
         // This only runs with `kotlin.dataframe.debug=true` in gradle.properties.
         if (BuildConfig.DEBUG) {
             require(values.all { it matches type }) {
-                val types = values.map { if (it == null) "Nothing?" else it!!::class.simpleName }.distinct()
+                val types = values.map { if (it == null) "Nothing?" else it::class.simpleName }.distinct()
                 "Values of $kind '$name' have types '$types' which are not compatible given with column type '$type'"
             }
         }

core/generated-sources/src/main/kotlin/org/jetbrains/kotlinx/dataframe/impl/schema/Utils.kt

@@ -116,7 +116,6 @@ internal fun ColumnSchema.createEmptyColumn(name: String): AnyCol =
         is ColumnSchema.Value -> DataColumn.createValueColumn<Any?>(name, emptyList(), type)
         is ColumnSchema.Group -> DataColumn.createColumnGroup(name, schema.createEmptyDataFrame()) as AnyCol
         is ColumnSchema.Frame -> DataColumn.createFrameColumn<Any?>(name, emptyList(), lazyOf(schema))
-        else -> error("Unexpected ColumnSchema: $this")
     }
 
 /**
@@ -146,8 +145,6 @@ internal fun ColumnSchema.createNullFilledColumn(name: String, numberOfRows: Int
             groups = List(numberOfRows) { emptyDataFrame<Any?>() },
             schema = lazyOf(schema),
         )
-
-        else -> error("Cannot create null-filled column of unexpected ColumnSchema: $this")
     }
 
 internal fun DataFrameSchema.createEmptyDataFrame(): AnyFrame =

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

@@ -43,7 +43,7 @@ class AllExceptTests : ColumnsSelectionDslTests() {
     }
 
     @Test
-    fun `exceptions`() {
+    fun exceptions() {
         shouldThrow<IllegalStateException> {
             dfGroup.select {
                 name.firstName.allColsExcept("firstName"["secondName"])

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

@@ -9,7 +9,7 @@ import org.junit.Test
 class FilterTests : ColumnsSelectionDslTests() {
 
     @Test
-    fun `filter`() {
+    fun filter() {
         listOf(
             df.select { cols(name, age, weight) },
             df.select { all().filter { "e" in it.name() } },

core/generated-sources/src/test/kotlin/org/jetbrains/kotlinx/dataframe/codeGen/ShortNamesRenderingTest.kt

@@ -78,7 +78,7 @@ internal class ShortNamesRenderingTest : TypeRenderingStrategy by ShortNames {
     }
 
     @Test
-    fun `dataframe`() {
+    fun dataframe() {
         fields.keys.asClue {
             fields["f"]!!.renderAccessorFieldType() shouldBe
                 "DataFrame<org.jetbrains.kotlinx.dataframe.internal.codeGen.ShortNamesRenderingTest.Marker>"

core/generated-sources/src/test/kotlin/org/jetbrains/kotlinx/dataframe/io/PlaylistJsonTest.kt

@@ -193,7 +193,7 @@ class PlaylistJsonTest {
     }
 
     @Test
-    fun `union`() {
+    fun union() {
         val merged = item.concat(item)
         merged.rowsCount() shouldBe item.rowsCount() * 2
         val group = merged.snippet

core/generated-sources/src/test/kotlin/org/jetbrains/kotlinx/dataframe/statistics/cumsum.kt

@@ -79,7 +79,7 @@ class CumsumTests {
     }
 
     @Test
-    fun `groupBy`() {
+    fun groupBy() {
         val df = dataFrameOf("str", "col")(
             "a", 1,
             "b", 2,

core/generated-sources/src/test/kotlin/org/jetbrains/kotlinx/dataframe/testSets/person/DataFrameTests.kt

@@ -257,7 +257,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `toMap`() {
+    fun toMap() {
         val map = df.toMap()
         map.size shouldBe 4
         map.forEach {
@@ -266,19 +266,19 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `size`() {
+    fun size() {
         df.size() shouldBe DataFrameSize(df.ncol, df.nrow)
     }
 
     @Test
-    fun `slicing`() {
+    fun slicing() {
         val sliced = typed[1..2]
         sliced.nrow shouldBe 2
         sliced[0].name shouldBe typed[1].name
     }
 
     @Test
-    fun `indexing`() {
+    fun indexing() {
         val expected = 45
         val i = 1
 
@@ -333,7 +333,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `chunked`() {
+    fun chunked() {
         val res = df.chunked(2)
         res.size() shouldBe 4
         res.toList().dropLast(1).forEach {
@@ -342,7 +342,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `update`() {
+    fun update() {
         fun AnyFrame.check() {
             getColumn(1).name() shouldBe "age"
             ncol shouldBe typed.ncol
@@ -423,7 +423,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `resetToNull`() {
+    fun resetToNull() {
         val updated = typed.update { all() }.withNull()
 
         updated.columns().forEach {
@@ -432,7 +432,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `sort`() {
+    fun sort() {
         val expected = listOf(null, "London", "Dubai", "Tokyo", "Milan", "Moscow", "Moscow")
 
         fun AnyFrame.check() = this[city].toList() shouldBe expected
@@ -472,7 +472,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `equals`() {
+    fun equals() {
         typed shouldBe typed.update { age }.with { age }
     }
 
@@ -504,7 +504,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `filter`() {
+    fun filter() {
         val expected = listOf("Bob", "Bob", "Charlie")
 
         fun AnyFrame.check() = this[name].toList() shouldBe expected
@@ -703,7 +703,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `groupBy`() {
+    fun groupBy() {
         fun AnyFrame.check() {
             nrow shouldBe 3
             this["name"].toList() shouldBe listOf("Alice", "Bob", "Charlie")
@@ -792,7 +792,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `min`() {
+    fun min() {
         val expected = 15
 
         fun Any?.check() = this shouldBe expected
@@ -848,7 +848,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `maxBy`() {
+    fun maxBy() {
         val expected = "Bob"
 
         fun AnyRow?.check() = this!![name] shouldBe expected
@@ -998,7 +998,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `distinct`() {
+    fun distinct() {
         val expected = 6
         typed.countDistinct { name and city } shouldBe expected
         typed.select { name and city }.distinct().nrow shouldBe expected
@@ -1018,7 +1018,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `addRow`() {
+    fun addRow() {
         val res = typed.append("Bob", null, "Paris", null)
         res.nrow shouldBe typed.nrow + 1
         res.name.type() shouldBe typeOf<String>()
@@ -1034,7 +1034,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `rename`() {
+    fun rename() {
         fun AnyFrame.check() {
             this["name2"].toList() shouldBe typed.name.toList()
             this["age2"].toList() shouldBe typed.age.toList()
@@ -1053,7 +1053,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `nunique`() {
+    fun nunique() {
         typed.name.countDistinct() shouldBe 3
     }
 
@@ -1486,7 +1486,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `forEachIn`() {
+    fun forEachIn() {
         val pivoted = typed.pivot(inward = true) { city }.groupBy { name and weight }.with { age }
         val sum = pivoted
             .select { "city".allCols() }
@@ -1497,7 +1497,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `parse`() {
+    fun parse() {
         val toStr = typed.convert { weight }.notNull { it.toString() }
         val weightStr = "weight".toColumnOf<String?>()
         val parsed = toStr.convert { weightStr }.toInt()
@@ -2190,7 +2190,7 @@ class DataFrameTests : BaseTest() {
     }
 
     @Test
-    fun `isNumber`() {
+    fun isNumber() {
         typed.age.isNumber() shouldBe true
         typed.weight.isNumber() shouldBe true
 

core/generated-sources/src/test/kotlin/org/jetbrains/kotlinx/dataframe/testSets/person/DataFrameTreeTests.kt

@@ -185,7 +185,7 @@ class DataFrameTreeTests : BaseTest() {
     }
 
     @Test
-    fun `selects`() {
+    fun selects() {
         df2.select { nameAndCity.allCols() } shouldBe typed2.nameAndCity.select { all() }
         df2.select { nameAndCity.cols { !it.hasNulls() } } shouldBe typed2.select { nameAndCity.name }
         df2.select { nameAndCity.cols(0..1) } shouldBe typed2.nameAndCity.select { all() }
@@ -237,47 +237,47 @@ class DataFrameTreeTests : BaseTest() {
     }
 
     @Test
-    fun `slice`() {
+    fun slice() {
         val expected = typed[0..2].name
         val actual = typed2[0..2].nameAndCity.name
         actual shouldBe expected
     }
 
     @Test
-    fun `filter`() {
+    fun filter() {
         val expected = typed.filter { city == null }.select { weight }
         typed2.filter { nameAndCity.city == null }.select { weight } shouldBe expected
         df2.filter { it[nameAndCity][city] == null }.select { weight } shouldBe expected
     }
 
     @Test
-    fun `select`() {
+    fun select() {
         val expected = typed.select { name and age }
         typed2.select { nameAndCity.name and age } shouldBe expected
         df2.select { it[nameAndCity][name] and age } shouldBe expected
     }
 
     @Test
-    fun `sort`() {
+    fun sort() {
         val expected = typed.sortBy { name and age }.moveTo(1) { city }
         typed2.sortBy { nameAndCity.name and age }.ungroup { nameAndCity } shouldBe expected
     }
 
     @Test
-    fun `move`() {
+    fun move() {
         val actual = typed2.move { nameAndCity.name }.into { pathOf("name") }
         actual.columnNames() shouldBe listOf("nameAndCity", "name", "age", "weight")
         actual.getColumnGroup("nameAndCity").columnNames() shouldBe listOf("city")
     }
 
     @Test
-    fun `groupBy`() {
+    fun groupBy() {
         val expected = typed.groupBy { name }.max { age }
         typed2.groupBy { nameAndCity.name }.max { age } shouldBe expected
     }
 
     @Test
-    fun `distinct`() {
+    fun distinct() {
         val duplicated = typed2.concat(typed2)
         duplicated.rowsCount() shouldBe typed2.rowsCount() * 2
         val dist = duplicated.nameAndCity.distinct()