Skip to content

Added logging to PodResult #112

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 52 commits into from
Oct 20, 2025
Merged

Added logging to PodResult #112

merged 52 commits into from
Oct 20, 2025

Conversation

@Synicix
Copy link
Contributor

@Synicix Synicix commented Aug 28, 2025 •
edited
Loading

Depends on #98 since it fixes the testing paradigm which is used by this code in testing.

Resolves PIPE-117

synicix and others added 30 commits August 4, 2025 23:38
Move to dev
844fdff
@Synicix
Remove uneeded results and change to return iterator
764387d
@Synicix
Remove accident comment
a03db37
@Synicix
Add ground work for pod operator
abbb5ce
@Synicix
Add async operation for JoinOperator
450a90f
@Synicix
Add pod operator + better error handling for pod_result:::new()
e4b3a54
@Synicix
Improve error handling for file io in localfilestore
fee8d9c
@Synicix
Update operator to be in operation again
c04335f
@Synicix
Fix pod result hashing bug
ebe04d1
@Synicix
Apply feedback from review
20f2345
@Synicix
Merge branch 'worktree-docker-patch' into 106
51dfea6
@Synicix
Remove stale error
cb72829
@Synicix
Merge branch 'worktree-docker-patch' into 106
d36ec71
@Synicix
Save changes
2986e0d
@Synicix
Merge branch '106' into pipeline_runner
0e3dc71
@Synicix
Split model in core to match uniffi
595a12e
@Synicix
Merge branch 'model_split' into 106
b671dba
@Synicix
Merge branch '106' into runner
ac363f2
@Synicix
Save progress
86d5f70
@Synicix
Merge branch 'dev' into model_split
aead411
@Synicix
Fix missing stuff
58ba198
@Synicix
Merge branch 'model_split' into runner
9c72bd8
@Synicix
save change
7ee5251
@Synicix
Update pipeline_runner to use new operator + improvments
fda43aa
@Synicix
Readded tests
1e9a19f
@Synicix
save progress
32299f8
@Synicix
Merge remote-tracking branch 'upstream' into runner
e93c32b
@Synicix
Fix logic bug
ab900b6
@Synicix
Update tests and test fixture to merge sentence correctly
4f511ce
@Synicix
Fix agent event to make it more efficient
2a4f1f7
@Synicix Synicix requested a review from eywalker August 28, 2025 09:55
@codecov
Copy link

codecov bot commented Aug 28, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 82.20472% with 113 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/core/pipeline_runner.rs 81.37% 92 Missing ⚠️
src/core/error.rs 12.50% 7 Missing ⚠️
src/uniffi/model/packet.rs 50.00% 6 Missing ⚠️
src/uniffi/model/pipeline.rs 64.28% 5 Missing ⚠️
src/uniffi/orchestrator/docker.rs 81.25% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@eywalker eywalker merged commit dfdd36b into nauticalab:dev Oct 20, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eywalker eywalker eywalker approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Synicix @eywalker