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

Adding locality parameter to python execution interface #1151

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

hkaiser
Copy link
Member

@hkaiser hkaiser commented Apr 24, 2020

  • flyby: adding sleep primitive

@taless474
Copy link
Contributor

@hkaiser, are changes on tests/unit/distributed/remote_add.cpp on purpose?

@hkaiser
Copy link
Member Author

hkaiser commented Apr 24, 2020

@hkaiser, are changes on tests/unit/distributed/remote_add.cpp on purpose?

Uhh, no. No idea how that happened - thanks for noticing.

@stevenrbrandt
Copy link
Member

stevenrbrandt commented Apr 28, 2020

The issue I have is this:

  • There's no way to send data from one locality to another inside a phylanx function
  • Arrays created in phylanx functions are deallocated when the function exits
  • Therefore all distributed array calculations must take place in a context where localities can't exchange data

This situation would be fixed if we had access to a global data structure inside phylanx functions. That way, data could be exchanged between invocations of phylanx functions.

@hkaiser hkaiser force-pushed the async_python branch 5 times, most recently from 223dc5a to e91cb62 Compare May 16, 2020 20:19
- add PHYLANX_WITH_PYTHON_TESTS_EMIT_FULL_DIAGNOSTICS cmake variable
- enable this option for CircleCI
- adding ability to add to the Phylanx session configuration through
  PHYLANX_PYTHON_CONFIG environment variable

- adding --hpx:ini=phylanx.full_error_diagnostics!=1 to enable full
  error diagnostics from Python code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants