Hi Henry, Thanks for the quick response, I appreciate it. I believe that I pulled the latest from master a day or so ago, so I’ll make sure to pull the most recent changes in.
As far as the placeholders, they aren’t currently present (as I don’t believe they were present in the Java ones within the learning/katas directory), however I can easily add those in to align with the content of the existing course. I wasn’t entirely sure based on the existing directories if the files should contain the placeholders or the actual implementations, either way, it’s a pretty trivial series of changes. I’ll try to put these together tomorrow and push up a PR. I’ll make sure to include you as a reviewer. Thanks for the initial feedback, Rion > On May 19, 2020, at 11:15 PM, Henry Suryawirawan <hsuryawira...@google.com> > wrote: > > > Thanks Rion for adding the Kotlin version. > This is great to show other people that Beam can be done in Kotlin too! > > I can help to review your work. > Please help to incorporate the Java Katas latest changes from master. > There are recent changes to the task description file format from html to md. > Please also help to remove all the *-remote-info.yaml files. > I assume that you've adjusted the answer placeholders in all tasks as well. > Afterwards, you can create a pull request and assign me as reviewer. > > Please reach out to me if you have any questions. > > > Regards, > Henry > > > > >> On Wed, May 20, 2020 at 3:33 AM Rion Williams <rionmons...@gmail.com> wrote: >> Sure! I ran through all of the tests locally on my branch (as tests) and >> then performed a check against all of the known tasks (via Course Creator > >> Check All Tasks) and 35/36 tasks passed successfully with the only one that >> didn't being a Built-in IO one that doesn't currently have any >> implementation. Although, I'd love for someone else to try the same thing >> since as far as I can tell it "works on my machine". >> >> Thanks! >> >> Rion >> >> On 2020/05/19 19:12:57, Pablo Estrada <pabl...@google.com> wrote: >> > This is really cool Rion! >> > >> > I believe it's possible to start trying out the katas from your branch? If >> > so, I can give them a try, and use that as a review... >> > Henry, any other ideas? >> > >> > On Tue, May 19, 2020 at 12:04 PM Rion Williams <rionmons...@gmail.com> >> > wrote: >> > >> > > Hi all, >> > > >> > > I was recently added as a contributor and created a JIRA ticket related >> > > to >> > > the existing Katas (https://issues.apache.org/jira/browse/BEAM-10027), >> > > specifically creating one that targets Kotlin specific as there are >> > > quite a >> > > few existing examples out there for Kotlin, so I thought a Kata course >> > > that >> > > would parallel the existing Java, Go, and Python ones. >> > > >> > > I basically ported over the existing Java Katas, added the appropriate >> > > dependencies, and converted all of the Java files over to Kotlin, and >> > > ensured that all of the tests pass as expected. I'd love outside of this >> > > to >> > > see if we can shift it to a Stepik course as well if that seems >> > > reasonable >> > > similar to those mentioned in this thread. >> > > >> > > My current branch awaiting a PR can be found here ( >> > > https://github.com/rionmonster/beam/tree/BEAM-10027), however I'm unsure >> > > who would be the best to review such a PR and what other steps might need >> > > to be taken before trying to get it merged in. >> > > >> > > Any feedback would be welcome! >> > > >> > > Thanks, >> > > >> > > Rion >> > > >> > > On 2020/05/14 23:40:45, Rion Williams <rionmons...@gmail.com> wrote: >> > > > +1 on the contributions front. My team and I have been working with >> > > > Beam >> > > primarily with Kotlin and I recently added the appropriate dependencies >> > > to >> > > Gradle and performed a bit of conversions and have it working as expected >> > > against the existing Java course. >> > > > >> > > > I don’t know how many others are actively working with Kotlin and Beam, >> > > but I’d love to work on transitioning that into a proper course (assuming >> > > there’s interest in it). >> > > > >> > > > > On May 14, 2020, at 10:32 AM, Nathan Fisher <nfis...@junctionbox.ca> >> > > wrote: >> > > > > >> > > > > >> > > > > Yes write IO >> > > > > >> > > > >> On Thu, May 14, 2020 at 05:41, Henry Suryawirawan < >> > > hsuryawira...@google.com> wrote: >> > > > >> Yeah certainly we can expand it further. >> > > > >> There are more lessons that definitely can be added further. >> > > > >> >> > > > >> >Eg more the write side windowing interactions? >> > > > >> Are you referring to Write IOs? >> > > > >> >> > > > >> >> > > > >> >> > > > >>> On Wed, May 13, 2020 at 11:56 PM Nathan Fisher < >> > > nfis...@junctionbox.ca> wrote: >> > > > >>> I went through them earlier this week! Definitely helpful. >> > > > >>> >> > > > >>> Is it possible to expand the katas available in the lO section? Eg >> > > more the write side windowing interactions? >> > > > >>> >> > > > >>>> On Wed, May 13, 2020 at 11:36, Luke Cwik <lc...@google.com> wrote: >> > > > >>>> These are an excellent learning tool. >> > > > >>>> >> > > > >>>>> On Tue, May 12, 2020 at 11:02 PM Pablo Estrada >> > > > >>>>> <pabl...@google.com> >> > > wrote: >> > > > >>>>> Sharing Damon's email with the user@ list as well. Thanks Damon! >> > > > >>>>> >> > > > >>>>>> On Tue, May 12, 2020 at 9:02 PM Damon Douglas < >> > > douglas.da...@gmail.com> wrote: >> > > > >>>>>> Hello Everyone, >> > > > >>>>>> >> > > > >>>>>> If you don't already know, there are helpful instructional tools >> > > for learning the Apache Beam SDKs called Beam Katas hosted on >> > > https://stepik.org. Similar to traditional Kata, they are meant to be >> > > repeated as practice. Before practicing the katas myself, I found myself >> > > copy/pasting code (Please accept my confession 😎 ). Now I find myself >> > > actually composing pipelines. Just like kata forms, you find them >> > > becoming >> > > part of you. If you are interested, below are listed the current >> > > available >> > > katas: >> > > > >>>>>> >> > > > >>>>>> 1. Java - https://stepik.org/course/54530 >> > > > >>>>>> >> > > > >>>>>> 2. Python - https://stepik.org/course/54532 >> > > > >>>>>> >> > > > >>>>>> 3. Go (in development) - https://stepik.org/course/70387 >> > > > >>>>>> >> > > > >>>>>> If you are absolutely brand new to Beam and it scares you like >> > > > >>>>>> it >> > > scared me, come talk to me. >> > > > >>>>>> >> > > > >>>>>> Best, >> > > > >>>>>> >> > > > >>>>>> Damon >> > > > >>> -- >> > > > >>> Nathan Fisher >> > > > >>> w: http://junctionbox.ca/ >> > > > > -- >> > > > > Nathan Fisher >> > > > > w: http://junctionbox.ca/ >> > > > >> > > >> >