Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add engineering OKRs for 2019Q1 #109

Merged
merged 6 commits into from
Mar 6, 2019
Merged

add engineering OKRs for 2019Q1 #109

merged 6 commits into from
Mar 6, 2019

Conversation

smola
Copy link
Contributor

@smola smola commented Feb 6, 2019

No description provided.

Signed-off-by: Santiago M. Mola <[email protected]>
README.md Outdated

## Hiring

- Hiring: Test (1 Senior) [P0][ENG]

Choose a reason for hiding this comment

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

No juniors for any team?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So far as OKR only for LA team. But we may still hire beyond these.

### Lookout Platform

- [ ] Collect GitHub reactions as feedback to comments [P1][APPS]
- [ ] Provide metrics on usage [P0][APPS]
Copy link

Choose a reason for hiding this comment

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

may we have info about which kind of metrics are meant here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@smacker We need to meet together with @vmarkovtsev and @rporres to define this. We need to define how analyzers should expose custom metrics (e.g. such as metrics about models built by style-analyzer).

### Simplify usage

- [ ] Gitbase to expose only reachable objects [P0][DP]
- [ ] Provide installers [P2][APPS][PRODUCT]
Copy link

Choose a reason for hiding this comment

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

installers of what? How is it different from engine that is an "installer" for most of src-d projects already?

Copy link
Contributor

Choose a reason for hiding this comment

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

I had the same doubt. Needs clarification.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Needs product discussion. This was initially meant to be apt-get and homebrew, but we still need to see if this would actually make things easier or not.

@@ -2,19 +2,134 @@

## Bring 'Code as Data' to market with enterprises

### Enable engine integration with 3rd parties

- [ ] gitbase-spark-connector SQL 3rd party interop (Spark Thrift Server) [P0][DP]
Copy link
Contributor

Choose a reason for hiding this comment

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

Since the section is "Enable engine integration with 3rd parties", will this be expected to be available in engine?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not now or not yet. Probably in next quarter, but still requires product discussions.

### Get bblfsh ready for wide language support of our use cases

- [ ] Make UAST v2 the standard interface for Babelfish [P0][LA]
- [ ] Compact diffs of UAST trees [P1][LA]
Copy link
Contributor

Choose a reason for hiding this comment

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

One thing that isn't clear in this document are the dependencies between teams that arise from these objectives. (I'm adding this comment here because this one happens to affect LA, but there are others).

I suppose some of that dependency information will probably fall out of the product planning cycle, but could we have the OKR document include some record of "shared" OKRs? E.g., we could record multiple teams, and/or assign numbers and say "required for 1.2.1" or something?

Signed-off-by: Santiago M. Mola <[email protected]>
### Improve current lookout analyzers

- Style Analyzer in one more language (possibly Python) [P0][ML]
- Release Typo Analyzer [P0][ML]
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • Style Analyzer improve the analysis quality [P1][ML]


- [ ] Collect GitHub reactions as feedback to comments [P1][APPS]
- [ ] Provide metrics on usage [P0][APPS]
- [ ] Live Demo Repository with all available analyzers set-up [P1][APPS][INFRA]
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • Define the analyzers configuration format in repositories [P1][?]

This is required to provide the rules blacklist feature in style-analyzer.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

material for the backlog at this point

Co-Authored-By: smola <[email protected]>
@bzz
Copy link
Contributor

bzz commented Mar 6, 2019

@smola do you think this should be merged at some point?

@smola smola merged commit b5dfe9a into 2019Q1 Mar 6, 2019
@smola smola deleted the 2019Q1-engineering branch March 6, 2019 18:43
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.

8 participants