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

Dependency updates and new package name #2839

Merged
merged 2 commits into from Oct 14, 2014
Merged

Dependency updates and new package name #2839

merged 2 commits into from Oct 14, 2014

Conversation

tschaub
Copy link
Member

@tschaub tschaub commented Oct 14, 2014

We now have access to the openlayers package name in the npm registry. I'd like to start publishing pre-releases there. Over time, we'll work out exactly what gets published. For now, it's the whole repo.

@elemoine
Copy link
Member

+1 (when Travis is happy again)

@tschaub
Copy link
Member Author

tschaub commented Oct 14, 2014

I get the same timeout locally. Looks like a genuine issue with the upgrade to proj4@2.3.3.

Will investigate a bit.

@tschaub
Copy link
Member Author

tschaub commented Oct 14, 2014

Actually, on master the same test takes 2.5s to run. Rather than bumping up the timeout, I'd like to determine out if the precision of our point resolution estimates is worth the computational intensity.

@tschaub
Copy link
Member Author

tschaub commented Oct 14, 2014

I was mistaken in thinking that the proj.getPointResolution method used the ellipsoid functions (of which I was suspicious in terms of intensity / increased precision). So to reduce the chance of test timeouts, I've proposed that we cut the number of cases (see #2840).

tschaub added a commit that referenced this pull request Oct 14, 2014
Dependency updates and new package name.
@tschaub tschaub merged commit 2b50734 into openlayers:master Oct 14, 2014
@tschaub tschaub deleted the publish branch October 14, 2014 20:44
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.

None yet

2 participants