ExaWorks / SDK

ExaWorks SDK
11 stars 12 forks source link

parsl-rp tutorial #156

Closed AymenFJA closed 1 year ago

AymenFJA commented 1 year ago

This PR adds the tutorial for the Parsl-RP integration related to #154.

review-notebook-app[bot] commented 1 year ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

mturilli commented 1 year ago

I had a chat with @kylechard about using this for the tutorial in February. We both agreed that the configuration should be loaded from a stand-alone file. At a later time and in another context, we will need to write a dedicated documentation page about how to configure the executor (probably to be added to Parsl's documentation) and then see how we can improve the configuration process by extending the standard Parsl configuration file.

AymenFJA commented 1 year ago

Most of the comments above are addressed now in the latest commit.

mtitov commented 1 year ago

@mturilli would it make sense to move parsl-rp notebook and its config file under the parsl directory? (SDK/docs/source/tutorials/parsl/)

mturilli commented 1 year ago

@mturilli would it make sense to move parsl-rp notebook and its config file under the parsl directory? (SDK/docs/source/tutorials/parsl/)

I would keep the tutorials under tutorial and use images, config, etc directories to store different types of ancillary files.

mtitov commented 1 year ago

I would keep the tutorials under tutorial and use images, config, etc directories to store different types of ancillary files.

that works! thanks!