Omkar P

Results 35 comments of Omkar P

Hey @neiledgar so I checked this out, and upon digging deeper I found that the issue here is that after the completion of `Updates()`, the model object (`&p` in your...

@neiledgar Thank you so much for your confirmation that the fix is working well. [The PR](https://github.com/go-gorm/gorm/pull/7099) is in review now (thanks to @a631807682) and when all concerns are resolved, it...

Hi @samuelrince, just wanted to highlight one thing I observed - the `ModelRepository.from_csv()` is loading the CSV file from disk each time it's called ([code here](https://github.com/genai-impact/ecologits/blob/main/ecologits/model_repository.py#L58-L60)), which is slowing down...

That's great to know @samuelrince! Let me know if I can help with any of the action items, would love to contribute.

> How does this work if partition is a date or a datetime? Would be nice with a test for this Thanks for the suggestion @thomasfrederikhoeck! Have added a test...

Hi @rtyler thanks for approving this PR. It seems that one of the concurrency tests for Windows is failing [here](https://github.com/delta-io/delta-rs/actions/runs/10532187167/job/29328798180?pr=2816#step:4:2022) with a `PermissionDenied` error. Please let me know if I...

> A test with a condition like `target.col in (source.col1, source.col2)` would be interesting Hi! I've added a Rust test that covers above mentiond condition [here in this PR](https://github.com/ion-elgreco/delta-rs/pull/2/files#diff-12f59fe3c4440b7ae4ee1a5ac810b42c1d7357c246aae7b5770e840e52d3ec52R1997-R2013). Also...

Yes! It's still accessible and comes up in top search engine results for "delta-rs docs". I stumble upon it every once in a while 😁 Probably we can consider a...

Yes the [old docs home page](https://delta-io.github.io/delta-rs/python/index.html) properly redirects to the new docs. But some other old docs pages (e.g. [API Reference](https://delta-io.github.io/delta-rs/python/api_reference.html)) still show up in top search results as they...

Seems like a nice feature to have! I can pick this up and raise a PR if no one is working on it, let me know.