Charles Stern

Results 434 comments of Charles Stern

Thanks @derekocallaghan. On first read, both the fix and the test look quite thorough and appropriate to me. I'm also unclear on what's causing the test failure. It seems possibly...

My hunch is that with some work the data should be loadable with `OpenWithKerchunk`. I'm not sure what that specific cause of the `KeyError` reported by @DarshanSP19 is off the...

Thanks for raising this, Julius. I think something like your pseudocode example is entirely possible in Beam. The challenge I see is that to deploy this, the pipeline object needs...

@rsignell-usgs, this question challenges some of my prior suggestions re: the direct runner. As we've observed, the direct runner appears to be performant in multithreaded mode, but on closer inspection...

👋 @rsignell-usgs thanks for checking in. Over in https://github.com/pangeo-forge/pangeo-forge-recipes/issues/618#issuecomment-1749287305 (and discussion continuing below there) we've been coming to terms with the fact that the multi-threaded runner does not appear to...

@hameer-spire, to clarify, are you encountering this when using `pangeo-forge-recipes`? If so, a bit more detail on your use case would be helpful. Thanks!

> You can make optional fields required in child classes, but you cannot make required fields optional. Perhaps this is not possible with `class`, however I think it is possible...

@FilipeMarch apologies, I somehow missed your ping initially, and now a huge amount of time has passed. I have not thought about this issue in a while, but yes, the...

Understood! To clarify, would you like `fastparquet` as a hard dependency right now? If so, happy to make the PR.

Hi @lzampier thanks for this contribution! I failing check is just a formatting hook. For future reference, you should be able to fix these issues yourself as described in https://github.com/pangeo-forge/staged-recipes/pull/230#issuecomment-1352443908....