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

[#29] Update readme to 0.6.0 #41

Merged
merged 11 commits into from
Sep 21, 2017

Conversation

spi-x-i
Copy link
Collaborator

@spi-x-i spi-x-i commented Sep 20, 2017

You can see preview of this here.
This closes #29

@spi-x-i spi-x-i added this to the 0.6.0 milestone Sep 20, 2017
README.md Outdated
and a *version*.

> **e.g.** Suppose you have two _ensemble_ models A and B (PMML based) where A has a depth level 10 on width
100 and B depth 5 on width 200, and you desire to have a comparison between them, so likely your can identify
Copy link
Collaborator

Choose a reason for hiding this comment

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

your -> you

README.md Outdated
`path` and a timestamp
- `DelMessage` it requires an `applicationName` Java UUID formatted, a `version` and a timestamp
- `DataStream[BaseEvent]` your input stream should extend the `BaseEvent` trait and defining the string `modelId
formatted as `"<modelApplication>_<modelVersion>"` and a timestamp.
Copy link
Collaborator

Choose a reason for hiding this comment

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

typo `

Copy link
Member

@maocorte maocorte left a comment

Choose a reason for hiding this comment

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

maybe architecture.png could be moved into a dedicated folder (for example images/) so it is not included in the builded jar

([repo](https://github.com/apache/flink)) should work properly.

## Adding `flink-jpmml` dependency
* if you employ sbt add the following dependecy to your project:
* Snapshot: `"io.radicalbit" %% "flink-jpmml-scala" % "0.6.0-SNAPSHOT"`
* Stable: `"io.radicalbit" %% "flink-jpmml-scala" % "0.5.1"`
* Snapshot: `"io.radicalbit" %% "flink-jpmml-scala" % "0.7.0-SNAPSHOT"`
Copy link
Member

Choose a reason for hiding this comment

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

Should be also updated the project version into version.sbt reflecting the changes

@maocorte maocorte merged commit bb957e1 into FlinkML:master Sep 21, 2017
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.

Updating README to version 0.6
3 participants