Skip to content

Field encoder - #763

Open
cchantep wants to merge 2 commits into
typelevel:masterfrom
cchantep:feature/field-encoder
Open

cchantep wants to merge 2 commits into
typelevel:masterfrom
cchantep:feature/field-encoder

Conversation

@cchantep

Copy link
Copy Markdown
Contributor

In case a type T encoding is supported in derivation (as a struct field), then this allows to resolve the corresponding TypedEncoder[T], so the field can be handled invidiually.

@pomadchin pomadchin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Though formatting is off (see CI)

Comment thread dataset/src/test/scala/frameless/TypedDatasetSuite.scala Outdated
"ds.col(_.a.toInt)" shouldNot typeCheck
}

test("col through record encoder (for Value class)") {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@cchantep
cchantep force-pushed the feature/field-encoder branch 4 times, most recently from 8306de0 to 6bf575d Compare September 27, 2026 20:31
@cchantep
cchantep force-pushed the feature/field-encoder branch from 6bf575d to 43bc905 Compare September 28, 2026 14:38

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants