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

Adding locality parameter to python execution interface #1151

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

Adding locality parameter to python execution interface #1151

wants to merge 3 commits into from

Conversation

hkaiser
Copy link
Member

@hkaiser hkaiser commented Apr 24, 2020

  • flyby: adding sleep primitive

@hkaiser hkaiser added this to the 0.0.1 milestone Apr 24, 2020
@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 •
edited
Loading

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
@hkaiser
Adding option to print full error information for HPX exceptions
04c5362 
- 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
Reviewers
No reviews
Assignees
No one assigned
Labels
category: @Phylanx Everything related to the @Phylanx python decorator category: primitives type: compatibility issue type: enhancement
Projects
None yet
Milestone
0.0.1
Development

Successfully merging this pull request may close these issues.
3 participants
@hkaiser @taless474 @stevenrbrandt