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

Add psij-python dependency for radical to avoid radical.saga layer #3079

Closed
wants to merge 1 commit into from

Conversation

benclifford
Copy link
Collaborator

See radical-cybertools/radical.saga#885 which repeatedly recommends avoiding the radical SAGA layer, and the corresponding Parsl issue #3029

Changed Behaviour

Job submission in the RadicalPilotExecutor will happen with a different codebase: psij-python, not radical.saga which will swap out radical.saga behavioural quirks for any psij-python ones.

Fixes

attempts to fix #3029

Type of change

  • Bug fix

See radical-cybertools/radical.saga#885
which repeatedly recommends avoiding the radical SAGA layer, and
the corresponding Parsl issue #3029

This PR updates the comment for psi-j-parsl in setup.py to reflect
the different between psi-j-parsl and psij-python.
@benclifford
Copy link
Collaborator Author

see that radical.saga issue radical-cybertools/radical.saga#885 for further discussion

@benclifford
Copy link
Collaborator Author

closing this pending further activity on #3029

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.

Test failures from Radical pilot executor
1 participant