Niels Bantilan
Niels Bantilan
thanks @riziles, let me digest this feedback. It might be time to do `pandera 1.0` and force users to install `pandas` so that it's not a core dependency. > Right...
In the mean time I'll look into fixing up #1704 to unblock this issue
fixed by #1904
@ragrawal I can test it out and see if we can simplify. Can you share full repro code on starting the server and making a call to the `/input_api` endpoint?
> It's just the removal of the Pandas dependency. Pandas is a heavy package that takes up a lot of space when spinning up environments and slows down start times...
Hi @schafsam, looks like the `tests/core/test_pandas_engine.py::test_pandas_datetimetz_dtype` is failing. > Adding the utc=True argument to the [to_datetime](https://pandas.pydata.org/docs/reference/api/pandas.to_datetime.html#pandas.to_datetime) function works with pandas and [modin.pandas.to_datetime](https://github.com/modin-project/modin/blob/main/modin/pandas/general.py) Can't this be addressed by adding `utc=True` to...
> I will give it a try, but need to figure out how or where it is set. If you provide a code snippet I can help out
hi @karajan1001 thanks for the contribution! Check out the contribution docs to learn how to run all the linters and tests locally to make sure things are working: https://pandera.readthedocs.io/en/stable/CONTRIBUTING.html
hi @karajan1001, you can find the parts of the code changes that were not covered in the tests in the "Files changed" tab, e.g. here: https://github.com/unionai-oss/pandera/pull/1227/files#diff-2691db6e54a8a776ed24713a7fa20bc1b86b6361e8a7bf7323722e301c359a76R1357
Okay, gonna try out SGD And report back with findings