Thanks, left a few comments, but overall this looks like great content! +1 to everything Robert said as well.
Thanks, Danny On Thu, Aug 15, 2024 at 9:25 PM Robert Burke <rob...@frantil.com> wrote: > I like it! > > My vote is put it on the beam site > > Likely linked from here > > https://beam.apache.org/documentation/ml/about-ml/ > > But also as a sibling to that page, it's in the python specific section at > least. > > If it's done in the next few days, before the cut it would be worth > including as a callout and link to the page in 2.59.0's CHANGES.md > > Website changes are technically outside of the release cycle, which make > updates harder to surface, but if we include them in CHANGES, they become > part of the release notes which geta better readership, even if the content > has been live longer. > > On Thu, Aug 15, 2024, 11:58 AM Jack McCluskey via dev <dev@beam.apache.org> > wrote: > >> Hey everyone, >> >> I've taken some time to write up some user-focused context around Beam >> Python's BatchElements transform. There are some nuanced considerations >> with this transform, especially the stateful implementation, and it felt >> like it was worth writing up! The end goal for the content is to be linked >> from the BatchElements docstring, either on this doc or on the >> documentation website proper. Feel free to take a look and give feedback! >> >> >> https://docs.google.com/document/d/1fOjIjIUH5dxllOGp5Z4ZmpM7BJhAJc2-hNjTnyChvgc/edit?usp=sharing >> >> Thanks, >> >> Jack McCluskey >> >> -- >> >> >> Jack McCluskey >> SWE - DataPLS PLAT/ Dataflow ML >> RDU >> jrmcclus...@google.com >> >> >>
