Parsl / parsl

Parsl - a Python parallel scripting library

Home Page:http://parsl-project.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Cannot Instantiate SlurmProvider in parsl-2023.12.11

Oddant1 opened this issue · comments

Describe the bug
When I attempt to instantiate a SlurmProvider in parsl-2023.12.11 I get the following error:
AttributeError: class SlurmProvider uses qos in the constructor, but does not define it as an attribute

To Reproduce
Steps to reproduce the behavior:

  1. Install parsl-2023.12.11
  2. Attempt to instantiate SlurmProvider
  3. See error

Expected behavior
The provider should be instantiated

Environment

  • OS: Seen on Ubuntu and MacOS github workers and Linux Mint 21.2
  • Python version: 3.8 and 3.11
  • Parsl version: 2023.12.11

Distributed Environment

  • I'm not actually running a script at all just trying to instantiate the provider/a config with the provider

Probably introduced by #2986 - maybe @tylern4 can look at this.

Basically for RepresentationMixin, each constructor argument needs to be stored on self so that it can be used to create repr output later.

This gets checked at instantiation, not statically, so it's more likely to happen for classes we don't test in CI.

Awesome, thank you @benclifford and @tylern4.