Skip to content

Add grpcio-tools#1552

Open
SohierDane wants to merge 1 commit into
mainfrom
grpcio_tools
Open

Add grpcio-tools#1552
SohierDane wants to merge 1 commit into
mainfrom
grpcio_tools

Conversation

@SohierDane

Copy link
Copy Markdown
Contributor

Add a dependency on grpcio-tools so we can recompile proto stubs in real time for kaggle_evaluation. It's a small library (<3 MB) and for my purposes the exact version doesn't much matter so I'm hoping this isn't a controversial addition. We do have alternatives if this addition isn't feasible but they're much more invasive, most likely migrating from gRPC to HTTP.

@SohierDane SohierDane requested a review from rosbo June 23, 2026 22:22
@Blevins42069

Copy link
Copy Markdown

#361

@rosbo

rosbo commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Can you add a simple smoke test under the tests forder for the grpcio-tools package for the method you are using to ensure we don't regress:

import unittest
import pandas as pd
class TestPandas(unittest.TestCase):
def test_read_csv(self):
data = pd.read_csv("/input/tests/data/train.csv")
self.assertEqual(100, len(data.index))

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