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 transformer entry point #116

Closed
wants to merge 5 commits into from
Closed

Add transformer entry point #116

wants to merge 5 commits into from

Conversation

christophe-rannou
Copy link

#96

@XciD
Copy link

XciD commented Nov 22, 2018

UP

…hub.com:christophe-rannou/jpmml-evaluator into feature/preprocessor-entry-point
@@ -66,6 +61,11 @@ private FieldValue declareMissing(FieldName name) {
return declare(name, (FieldValue) null);
}

@Override
protected FieldValue prepare(FieldName name, Object value) {
throw new NotImplementedException();
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 java.lang.UnsupportedOperationException instead.

The org.jpmml.evaluator.NotImplementedException has a different meaning - "the PMML specification defines some feature/nehaviour, but this is not yet implemented in the JPMML-Evaluator library"

Copy link

Choose a reason for hiding this comment

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

Fixed

@vruusmann
Copy link
Member

What's the use case for 64-bit integer values? Counting milliseconds since 01-01-1970?

Please comment here: #141

Adrien Carreira and others added 3 commits January 30, 2019 09:06
@vruusmann
Copy link
Member

Duplicate of #96

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