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

[Go] ai.Model is a separate type from its action #458

Merged
merged 1 commit into from
Jun 24, 2024
Merged

Conversation

jba
Copy link
Contributor

@jba jba commented Jun 24, 2024

This continues the work begun in #402 of making the main ai types
distinct from their underlying actions, instead of aliases.

This allows the types to have methods, unstead of using top-level
functions.

It also clarifies documentation and other output, like panic stack
traces.

@jba jba force-pushed the jba/rename-definestore branch from 442ca56 to 201f427 Compare June 24, 2024 14:58
Base automatically changed from jba/rename-definestore to main June 24, 2024 15:05
This continues the work begun in #402 of making the main ai types
distinct from their underlying actions, instead of aliases.

This allows the types to have methods, unstead of using top-level
functions.

It also clarifies documentation and other output, like panic stack
traces.
@jba jba force-pushed the jba/model-unalias branch from 329c99f to ded8b3d Compare June 24, 2024 15:06
@jba jba merged commit 0db5280 into main Jun 24, 2024
5 checks passed
@jba jba deleted the jba/model-unalias branch June 24, 2024 15:25
jba added a commit that referenced this pull request Jun 24, 2024
jba added a commit that referenced this pull request Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants