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 pydantic classes for output from fusion callers #228

Merged
merged 11 commits into from
Jan 23, 2025
Merged

Conversation

jarbesfeld
Copy link
Contributor

closes #227

This PR creates pydantic classes for each fusion caller in a new file called fusion_caller_models.py. These classes are then called in translator.py

@jarbesfeld jarbesfeld added the priority:medium Medium priority label Jan 16, 2025
@jarbesfeld jarbesfeld marked this pull request as ready for review January 16, 2025 19:56
@jarbesfeld jarbesfeld self-assigned this Jan 16, 2025
@jarbesfeld jarbesfeld requested a review from korikuzma January 16, 2025 19:56
Copy link

This PR is stale because it has been open 3 day(s) with no activity. Please review this PR.

@github-actions github-actions bot added the stale label Jan 20, 2025
@korikuzma korikuzma removed the stale label Jan 20, 2025
Base automatically changed from issue-224 to main January 21, 2025 19:37
src/fusor/fusion_caller_models.py Outdated Show resolved Hide resolved
src/fusor/fusion_caller_models.py Outdated Show resolved Hide resolved
src/fusor/fusion_caller_models.py Outdated Show resolved Hide resolved
src/fusor/fusion_caller_models.py Outdated Show resolved Hide resolved
src/fusor/translator.py Outdated Show resolved Hide resolved
@jarbesfeld jarbesfeld requested a review from korikuzma January 22, 2025 14:16
@jarbesfeld jarbesfeld requested a review from korikuzma January 23, 2025 14:20
@jarbesfeld jarbesfeld merged commit f16f018 into main Jan 23, 2025
10 checks passed
@jarbesfeld jarbesfeld deleted the issue-227 branch January 23, 2025 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create pydantic models for the different fusion callers
2 participants