Skip to content

Conversation

@bzz
Copy link
Member

@bzz bzz commented Dec 16, 2019

Small updates:

  • update all image names
  • slides link TBD
  • add notebook links that work on GH as well
  • update workshop description to match the published one

Signed-off-by: Alexander Bezzubov <bzz@apache.org>
@bzz bzz requested a review from m09 December 16, 2019 13:31
Copy link
Contributor

@m09 m09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor things, LGTM otherwise

> During this 3.5 hours workshop, we will review the recent SE tasks that benefit from applying ML and focus the hands-on experience on:
> - extracting data from the real source code and
> - developing multiple different ML models
> - for a particular task of source code summarization (or function name suggestion).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
> - for a particular task of source code summarization (or function name suggestion).
> - summarizing source code, suggesting function names and analyzing projects and developers most relevant topics

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So far this text was 1:1 copy of the materials published on the website.
I'm not sure what's best to do in here

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's probably fine to improve it here but I'm also fine keeping it as is.

Apply review feedback

Co-Authored-By: m09 <142691+m09@users.noreply.github.com>
Copy link
Contributor

@m09 m09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should probably modify the Dockerfile too to keep the README synced with the rest of the repo (for the paths that we mount for example), and maybe some other files? Not sure about that.

Signed-off-by: Alexander Bezzubov <bzz@apache.org>
@bzz
Copy link
Member Author

bzz commented Dec 16, 2019

We should probably modify the Dockerfile too to keep the README synced with the rest of the repo (for the paths that we mount for example), and maybe some other files? Not sure about that.

Tnx, done in 3efc1c3. The root dir name is not changed yet.

@bzz bzz merged commit c21ed3f into master Dec 20, 2019
@bzz bzz deleted the doc-fixup branch December 20, 2019 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants