Skip to content

chore: update to 2.18 and add prediction_log.proto#23

Merged
figroc merged 2 commits intofigroc:masterfrom
scurkovic:master
Feb 6, 2025
Merged

chore: update to 2.18 and add prediction_log.proto#23
figroc merged 2 commits intofigroc:masterfrom
scurkovic:master

Conversation

@scurkovic
Copy link

@scurkovic scurkovic commented Jan 28, 2025

I needed the prediction_log.proto to be able to generate the TFRecord for the Tensorflow Serving Warmup feature. At the same time, it bumps the TF version to 2.18.

@figroc
Copy link
Owner

figroc commented Jan 28, 2025

Thanks for your PR. I'll look into the failed CI in following couple days and merge it.

@scurkovic scurkovic changed the title chore: update to 2.17 and add more protos chore: update to 2.18 and add more protos Jan 29, 2025
@scurkovic
Copy link
Author

Sorry, it's potentially my bad as I made the proto glob too permissive. I've reverted it to the original glob and only added that single prediction_log.proto file. Hopefully it will pass now.

@scurkovic scurkovic changed the title chore: update to 2.18 and add more protos chore: update to 2.18 and add prediction_log.proto Jan 29, 2025
@mb-ii
Copy link

mb-ii commented Feb 5, 2025

Hey there @figroc! Any possibility you review this PR this week or early next week? This would be useful for my team as well. Thanks!

@figroc
Copy link
Owner

figroc commented Feb 6, 2025

Hey there @figroc! Any possibility you review this PR this week or early next week? This would be useful for my team as well. Thanks!

There are 2 proto files missing in the PR. I'm looking into it.

@figroc
Copy link
Owner

figroc commented Feb 6, 2025

I'll merge this PR and patch the issues shortly.

@figroc figroc merged commit ca6ba4a into figroc:master Feb 6, 2025
1 check failed
@figroc figroc mentioned this pull request Feb 6, 2025
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.

3 participants

Comments