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

Incredibly slow build times using Turkserver with Meteor 1.2 #54

Closed
ejolly opened this issue Oct 14, 2015 · 5 comments
Closed

Incredibly slow build times using Turkserver with Meteor 1.2 #54

ejolly opened this issue Oct 14, 2015 · 5 comments
Labels

Comments

@ejolly
Copy link

ejolly commented Oct 14, 2015

I've seen that some people have generally experienced issues with slower build times for apps after upgrading to Meteor 1.2, but any app I have that includes Turkserver is incredibly slow to build in comparison to apps that don't include Turkserver with the latest version of both Meteor and Turkserver. Are there any suggestions for dealing with this (e.g. package add options, removing irrelevant packages), or do we simply have to wait until the build speed issues are taken care of on the Meteor side of things?

@mizzao
Copy link
Member

mizzao commented Oct 14, 2015

I was thinking about transitioning the MTurk API to use https://github.com/JoseBarrios/mturk-api, which would get rid of a lot of the outdated binary dependencies clogging up the build process, but haven't gotten around to that yet.

Happy to take any contributions or ideas you may have, though.

@mizzao
Copy link
Member

mizzao commented Nov 13, 2015

How slow is your build exactly?

@ejolly
Copy link
Author

ejolly commented Nov 16, 2015

I updated to 1.2.1 about a week ago and didn't measure build times with the 1.2.0 release. Currently it's a tiny bit faster with that update, but I'm averaging about 1 minute 15 seconds. In versions < 1.2, the build time was never more than 30 seconds.

@mizzao
Copy link
Member

mizzao commented Nov 18, 2015

Hey @ejolly, I started a new branch using https://github.com/JoseBarrios/mturk-api: https://github.com/HarvardEconCS/turkserver-meteor/tree/new-api

I think the build process here should be much faster, but the MTurk API looks a little different and hence some of the functions might be broken and need fixing. But if you find building to be faster, then we're on the right track.

Once I can confirm everything works on this branch, I'll merge it into master.

@mizzao
Copy link
Member

mizzao commented Dec 14, 2015

As mentioned at #24 (comment), things might be much faster for you after switching to the new underlying MTurk API. Let me know if the build is smoother.

@mizzao mizzao closed this as completed Jul 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants