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.

Sign up for GitHub

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

Jump to bottom

Remove perspective.node Python module. #912

Merged
merged 1 commit into from
Feb 9, 2020
Merged

Remove perspective.node Python module. #912

merged 1 commit into from
Feb 9, 2020

Conversation

texodus
Copy link
Member

@texodus texodus commented Feb 8, 2020

Remove perspective.node Python module.

  • It is slower than native Python build, both its pure runtime performance and data marshal costs.
  • It has a quirky weird API, which is wholly deprecated by the native Python library.
  • It has obstinate memory limitations in 64bit.
  • The native node extensions are a pain and inflict specific node.js version compatibility issues through all lerna workspaces Bump core-js to v3.6.4 and Babel to 7.8.4 #906 updates and fixes for windows build #884

@texodus texodus added enhancement Feature requests or improvements Python labels Feb 8, 2020
@texodus texodus merged commit ff1a34d into master Feb 9, 2020
@texodus texodus deleted the remove-python-node branch February 9, 2020 03:18
@texodus texodus mentioned this pull request Feb 20, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@timkpaine timkpaine timkpaine approved these changes

Assignees
No one assigned
Labels
enhancement Feature requests or improvements Python
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@texodus @timkpaine @finos-admin