-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ColumnsSelection DSL overhaul #372
Merged
Merged
Changes from 167 commits
Commits
Show all changes
169 commits
Select commit
Hold shift + click to select a range
6511a50
valueColumn concept start
Jolanrensen 65fbbb6
Merge branch 'dfs-rename3' into continued-columnsselectiondsl
Jolanrensen fb37b27
valueCols, frameCols and tests with recursively
Jolanrensen b60f1fd
updating docs processor version. colsOfKind test, writing docs for co…
Jolanrensen a7fa80b
col(index) now also throws IndexOutOfBoundsException, working on docs
Jolanrensen 4440d20
cols(indices) docs finished
Jolanrensen e1095de
docs for valueCols, colGroups, frameCols
Jolanrensen 8da2fe2
added missing overloads for all flavors
Jolanrensen 1cbc563
added docs for allFrom, allAfter, allBefore, allUpTo
Jolanrensen 9273479
added check function to provide helpful error messages when using non…
Jolanrensen 1069b08
universal columnGroup check for SingleColumn
Jolanrensen 2bc3902
added more colGroup checks, working on select
Jolanrensen efc5a48
new parent for ColumnSet and SingleColumn: ColumnsResolver. So, SingC…
Jolanrensen 478485b
added missing docs for ColumnPath overloads, docs for select {}
Jolanrensen 1d04d28
overloads for roots, children, take, drop (and variants)
Jolanrensen 3ac16ae
children {}
Jolanrensen 592d060
start with `and`, added invoke overloads for select {} tbd
Jolanrensen 4892851
updated select docs, "and" docs
Jolanrensen fb19a4e
drop(Last) and take(Last) docs
Jolanrensen 2eec84a
fixed some usages of KProperty api in docs. added docs for take/drop(…
Jolanrensen 2d8887b
overhaul for references in csdsl. Always specifiying receiver type if…
Jolanrensen 4d996bb
updated doc processor version, tests, filter {}, overloads for nameCo…
Jolanrensen 8a322af
tests and docs for nameContains
Jolanrensen e603e4d
updated docs for first, single, last. Added firstChild etc. overloads…
Jolanrensen 357c6f1
overloads for except and allExcept, WIP
Jolanrensen 4faa62c
docs for nameStartsWith/nameEndsWith, distinct, overloads and docs fo…
Jolanrensen deedcad
named/into docs and overloads
Jolanrensen ab56cf8
asColumnGroup() overloads, expanded * imports across the library
Jolanrensen d9f4026
untyped String.invoke(), fixed allColumnsExceptKeepingStructure
Jolanrensen 4d1d440
linting
Jolanrensen 33e41db
added selection notebook to ask for feedback
Jolanrensen 9799143
added selection notebook to ask for feedback
Jolanrensen 3100845
small notebook update
Jolanrensen 59f618b
small doc fix
Jolanrensen a3c703c
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen 67e1dfe
initial ColumnsSelectionDsl/SingleColumn split. Needs more tests and …
Jolanrensen 073c196
moved first, single, last to their own files
Jolanrensen e48d072
split off all, children, colGroups, cols, colsOfKind, constructors, f…
Jolanrensen 5ca56fe
split off drop and take functions, fixed linting
Jolanrensen 36ba0a7
split off except, withoutNulls, column name filters
Jolanrensen f1a2569
split off and, colsOf, range of columns, ColumnSelectionDsl, asColumn…
Jolanrensen 4ebea6f
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen 436dcd2
update version
Jolanrensen b2bc962
readying first, last, and single for reviews
Jolanrensen 5ebe712
processing review, renamed Child to Col for first, last, and single
Jolanrensen 5c5751c
added todo message for allExcept in this PR
Jolanrensen b202ad1
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen 8d86761
update from master
Jolanrensen a5fd642
moved col() from constructors to its own file, expanded and added ove…
Jolanrensen 28e0134
fixed col on singleColumn, added new tests for col
Jolanrensen 620f577
fixed col on singleColumn, added new tests for col
Jolanrensen ae2acd5
Merge remote-tracking branch 'origin/continued-columnsselectiondsl' i…
Jolanrensen bfb2348
fixed bug where columnGroup("nonGroup").column("something") would be …
Jolanrensen caefdf3
exception tests for col
Jolanrensen ffeba3e
valueCol overloads and moved into its own file
Jolanrensen d24ef2f
performCheck function for ColumnAccessors. valueCol tests
Jolanrensen 1129fbb
frameCol overloads and tests
Jolanrensen e6bc06a
colGroup overloads and tests, removed interface from constructors.kt
Jolanrensen 053155f
updates were made to the documentation in the constructors.kt file. T…
Jolanrensen 9766997
updated cols.kt file with overloads. still need to check and remove s…
Jolanrensen cd828a9
moved some get operator functions to extension functions for DataFram…
Jolanrensen 70e9c8d
removed cols get operations with indices on non-columnSets. fixed col…
Jolanrensen 096d32c
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen f24d8b8
updating from master, replacing deprecated @tags
Jolanrensen 1a39e46
updating some ? notation based on feedback
Jolanrensen c57e9ee
updating columnRange and its tests
Jolanrensen 2edf8ef
changing ensureIsX() error messages.
Jolanrensen c284c2a
working on DSL grammars
Jolanrensen 50d2063
updating DSL grammars with **bold** notation
Jolanrensen bb6da80
DSL grammar up till col done
Jolanrensen b1ef8cf
DSL grammar up till colGroup done
Jolanrensen af7dce7
a bit of structure in dsl grammars
Jolanrensen 74fc3dd
DSL grammar up till cols done
Jolanrensen 403a9b4
removed asColumnGroup overloads except for SingleColumn which was mov…
Jolanrensen 0f4539f
column range done, expanded usage template
Jolanrensen e1abe4a
added recursively to dsl grammars where missing, fixed cols grammar, …
Jolanrensen 3569f57
removed unnecessary KProperty receiver overloads
Jolanrensen f548cd9
added column path overloads for cols and optional typing for string-a…
Jolanrensen 6383d64
moved usageTemplate, done until colGroups
Jolanrensen aae13c4
done up until frameCols, also update DSL grammar in ColumnsSelectionDsl
Jolanrensen e25e799
colsOfKind done
Jolanrensen eb8b761
reworking all-family of functions CS-DSL
Jolanrensen cb9d80b
added deprecated all-family functions
Jolanrensen 1100d93
continuing all-family of functions CS-DSL, started writing tests, add…
Jolanrensen dea2241
renamed performCheck to onResolve. added and fixed overloads for all-…
Jolanrensen e85b75c
updated cs-dsl usage
Jolanrensen 8fcec64
finished all functions, moved logic to single place, fixed references…
Jolanrensen d8e2cf6
renamed recursively/rec to atAnyDepth and deprecated original
Jolanrensen 02391a4
experimenting with scope-notation atAnyDepth
Jolanrensen 8e1b7c0
children->colsInGroups. Avoiding parent/child names in column groups.…
Jolanrensen a8d332d
operation.atAnyDepth() -> colsAtAnyDepth().operation, for simplicity,…
Jolanrensen 464b413
colsAtAnyDepth {} kdocs updates and references to the old name
Jolanrensen 100e145
removing references to .atAnyDepth() in examples in kdocs
Jolanrensen cb8a51c
updated docs with korro
Jolanrensen 92b6d87
more renaming of public `children()` functions to `cols()` to drop an…
Jolanrensen bf93dbe
take(Last)(Cols)(While) done
Jolanrensen 6e03cfa
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen beb48cf
drop(Last)(Cols)(While) done
Jolanrensen 7e14263
Extra information for Column not found error in DataFrameReceiver. Cl…
Jolanrensen 2d09166
removed references to transformable columnsets
Jolanrensen 71dc5b0
done with refactoring ColumnNameFilters
Jolanrensen 7b3f476
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen 677c3bf
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen 3122362
updated from master post 0.12 release
Jolanrensen 0c6069c
moved new deprecated methods to 0.13 warning region
Jolanrensen 5d3eaae
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen 50112fc
bumping simple-git version
Jolanrensen aec1386
moved select functions outside of the ColumnsSelectionDsl where possible
Jolanrensen 9f82253
withoutNulls, renamed, added usage, tests
Jolanrensen c5f34e0
distinct, added usage and tests. Added example for CS dsl UsageTemplate
Jolanrensen dbaa5a9
none, added usage and tests.
Jolanrensen abfb528
reworked colsOf, added tests and usage
Jolanrensen 6bf8a04
added transformSingleWithContext, experiments for except. renamed roo…
Jolanrensen 1452405
added usage for simplify and moved some experiments
Jolanrensen ff15430
removed testing notebooks
Jolanrensen eff8fbb
improve none() test
Jolanrensen 658f0e8
and function docs, usage, tests
Jolanrensen 8a242c8
rename/into added usage and tests
Jolanrensen 349c356
Deprecated List<DataColumn<C>>[IntRange] because there were no usages…
Jolanrensen 381d514
moved expr {} and started with Usage and tests
Jolanrensen ed40fa2
weakened type in mapToColumn from ColumnAccessor to ColumnReference s…
Jolanrensen e536b48
finished tests for expr in selection dsl
Jolanrensen 0764e48
Changed ColumnSet.allX() functions to take ColumnFilter instead of Co…
Jolanrensen 5820e40
removed overloads for filter aside from ColumnSet, wrote tests and Usage
Jolanrensen 5979821
updating from master
Jolanrensen 9113dc1
fixing linting
Jolanrensen 0c8ad63
Added @DslMarker for CS-DSL such that accessors work better across sc…
Jolanrensen 7453335
"fixed" KProperty<DataRow<T>>.select {} and KProperty<T>.select clash…
Jolanrensen bff2089
added select usage
Jolanrensen d9c6948
kdoc updates
Jolanrensen a0c33ba
added ! to DataColumnArithmetics.kt
Jolanrensen 04c0e56
cols(vararg) can now throw IllegalArgumentException for non-existing …
Jolanrensen 022e7b1
wip allColsExcept and fixing errors which show ColumnPaths
Jolanrensen 9e433d5
wip allExcept. Added helper function for ColumnPath to drop its start…
Jolanrensen c4b3da7
removed allColsExcept infix overloads
Jolanrensen 9ed12ed
Applying KT-64092 temp-fix to all(Cols)X functions as well
Jolanrensen 35e4e19
Applying KT-64092 temp-fix to all(Cols)X functions as well
Jolanrensen 681be27
updating error messages with column paths
Jolanrensen 7f96d6f
improving except api and writing tests
Jolanrensen ee69a5d
improving except api and writing tests
Jolanrensen c669c06
working on kdocs, empty groups are now removed in new except functions
Jolanrensen 8c81454
except usage grammar
Jolanrensen 796f74d
small `and` changes simplifying scopes. WIP kdocs for except
Jolanrensen 21bbbed
simplifying `and`, continuing kdocs `except`, adding experimental `ex…
Jolanrensen af637fb
minor doc fix
Jolanrensen d6a2d1c
working on except docs
Jolanrensen 6fac7f1
finished non-experimental except docs
Jolanrensen 87356c0
except kdocs done
Jolanrensen 8bd09c9
sorted CSDSL
Jolanrensen 4945870
updated Usage links across CsDSL
Jolanrensen 4180389
updating kdocs
Jolanrensen 18c5db4
fixed documentation urls
Jolanrensen 25e1b56
kdocs created for DataFrame.select.
Jolanrensen 8a1b557
dsl grammars fixed and added explanation with links to it
Jolanrensen 17def06
linting
Jolanrensen fc83336
KType type def fixed in kdoc
Jolanrensen b439919
comment fix
Jolanrensen 050f794
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen 9590e3b
update from master
Jolanrensen 94e6f87
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen be1e9e9
improving kdocs all across the CS DSL: simplifying first line for cla…
Jolanrensen f228165
improving kdocs for typealiases for lambdas, linting
Jolanrensen 12f620a
renaming all "Usage" docs to "Grammar"
Jolanrensen 721b390
adding note regarding return type of lambda CS-DSL
Jolanrensen 46a18d5
adding something to each CS dsl interface
Jolanrensen 7b54184
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen b99f3f4
updating from master
Jolanrensen 9450590
fixed documentation notes based on review
Jolanrensen 31c71c3
Merge branch 'master' into continued-columnsselectiondsl
Jolanrensen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
26 changes: 23 additions & 3 deletions
26
core/generated-sources/src/main/kotlin/org/jetbrains/kotlinx/dataframe/DataColumn.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 changes: 2 additions & 2 deletions
4
.../generated-sources/src/main/kotlin/org/jetbrains/kotlinx/dataframe/aggregation/aliases.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,10 @@ | ||
package org.jetbrains.kotlinx.dataframe.aggregation | ||
|
||
import org.jetbrains.kotlinx.dataframe.Selector | ||
import org.jetbrains.kotlinx.dataframe.columns.ColumnSet | ||
import org.jetbrains.kotlinx.dataframe.columns.ColumnsResolver | ||
|
||
public typealias AggregateBody<T, R> = Selector<AggregateDsl<T>, R> | ||
|
||
public typealias AggregateGroupedBody<G, R> = Selector<AggregateGroupedDsl<G>, R> | ||
|
||
public typealias ColumnsForAggregateSelector<T, C> = Selector<ColumnsForAggregateSelectionDsl<T>, ColumnSet<C>> | ||
public typealias ColumnsForAggregateSelector<T, C> = Selector<ColumnsForAggregateSelectionDsl<T>, ColumnsResolver<C>> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will be in the next release of the doc preprocessor, I must not forget to enable it then :)