improve comment

hyparam · Jan 14, 2025 · 00c80d1 · 00c80d1
1 parent 7d138f6
commit 00c80d1
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/dataframe.ts b/src/dataframe.ts
@@ -118,7 +118,8 @@ export function sortableDataFrame(data: DataFrame): DataFrame {
     rows(start: number, end: number, orderBy?: string): AsyncRow[] | Promise<Row[]> {
       if (orderBy) {
         if (!data.header.includes(orderBy)) {
-          /// TODO(SL): should we allow '__index__'?
+          // '__index__' is not allowed, and it would not make sense anyway
+          // as it's the same as orderBy=undefined, since we only support ascending order
           throw new Error(`Invalid orderBy field: ${orderBy}`)
         }
         if (!all) {