Skip to content

Conversation

@yaronskaya
Copy link
Contributor

No description provided.

@yaronskaya yaronskaya requested a review from astansler October 24, 2017 14:37
var pklObject = PickleUtil.unpickle(storage)
val pipeline = pklObject as Pipeline
pklObject = PMMLPipeline().setSteps(pipeline.getSteps())
val pmml = pklObject.encodePMML()
Copy link
Member

Choose a reason for hiding this comment

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

Initial loading of js evaluator takes 3 minutes on my machine, it's ok for first launch, but would be better if we can optimize time for subsequent launches. What if we generate PMML files from pickle on the first launch and then use PMML files to instantiate evaluators, will it be faster?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If one uses not all languages - it might lead to 200 mb as before.

Copy link
Member

Choose a reason for hiding this comment

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

I think that it's ok to generate such files locally, the problem was with transmission of such big files via internet. Also, I'm suggesting to generate evaluators on demand.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think that it's ok to generate such files locally, the problem was with transmission of such big files via internet.

I understand.

Also, I'm suggesting to generate evaluators on demand.

I thought we already do that..

Copy link
Member

@astansler astansler Oct 25, 2017

Choose a reason for hiding this comment

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

And cache it to PMML files for subsequent launches, see first comment

@astansler astansler merged commit 9c2d2cf into develop Oct 25, 2017
@astansler astansler deleted the develop-oxxxymiron branch October 25, 2017 18:45
astansler pushed a commit that referenced this pull request Nov 27, 2017
* fix: extract lib lines from diff only

* chore: swap nums in lib stats

* fix: extract lib lines from diff only

* chore: swap nums in lib stats

* feat: load models from *.pkl.z directly

* Add external lib

* feat: cach to pmml file

* wip: add handling exceptions while caching

* wip: more exceptions

* chore: fix 80 column limit, fix log messages

* chore: remove models.zip, add more logs
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