Skip to content
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

feat: add Text.from(Var)Array #674

Merged
merged 8 commits into from
Dec 4, 2024
Merged

feat: add Text.from(Var)Array #674

merged 8 commits into from
Dec 4, 2024

Conversation

ggreif
Copy link
Contributor

@ggreif ggreif commented Dec 3, 2024

Using fromIter is probably less-than-optimal performance-wise, but we can add a compiler primitive if it proves to become a bottleneck. For now it fills a hole that I perceived in the Motoko AI hackathon.

Adds test for both Text.fromArray and fromVarArray.

@ggreif ggreif self-assigned this Dec 3, 2024
@ggreif ggreif added the enhancement New feature or request label Dec 3, 2024
@ggreif ggreif requested review from crusso and rvanasa December 3, 2024 21:52
crusso
crusso previously approved these changes Dec 4, 2024
Copy link
Contributor

@crusso crusso left a comment

Choose a reason for hiding this comment

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

but maybe see what @rvanasa thinks too!

Copy link
Contributor

@rvanasa rvanasa left a comment

Choose a reason for hiding this comment

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

LGTM!

@ggreif ggreif enabled auto-merge (squash) December 4, 2024 15:29
@ggreif ggreif merged commit 6d4caef into master Dec 4, 2024
6 checks passed
@ggreif ggreif deleted the gabor/fromArray branch December 4, 2024 15:33
ggreif added a commit that referenced this pull request Dec 4, 2024
Using `fromIter` is probably less-than-optimal performance-wise, but we
can add a compiler primitive if it proves to become a bottleneck. For
now it fills a hole that I perceived in the Motoko AI hackathon.

Adds test for both `Text.fromArray` and `fromVarArray`.
ggreif added a commit that referenced this pull request Dec 4, 2024
Using `fromIter` is probably less-than-optimal performance-wise, but we
can add a compiler primitive if it proves to become a bottleneck. For
now it fills a hole that I perceived in the Motoko AI hackathon.

Adds test for both `Text.fromArray` and `fromVarArray`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants