Ali Khosravi
Ali Khosravi
> Yes, and? How does that contradict what I said? I really don't understand how else to explain it. We are running identical tests, twice.
@GeigerJ2 have you considered aligning this fixtures with the already existing fixture: https://github.com/aiidateam/aiida-core/blob/0eb794e1d1f584c78f7a6eddf908158b2f3848ce/tests/cmdline/commands/test_node.py#L579-L598 I think that would make a more concise "fixture-suite" in out code-base.
Thanks @agoscinski I write things here, only not to forget - Agreed, the process doesn't seem to be completely taken off from the concurrent Q, it's still there until its...
> So we would really have to check whether the job was already in EBM before the kill call, and only then ignore any exceptions and not go in the...
Don't worry about tests failing. They require https://github.com/aiidateam/plumpy/pull/288 to function.. which I couldn't figure out how to properly hook it as dependency.
Thanks @sphuber, you are right, that actually makes sense... yeah, I don't remember why I did it this way, will apply the changes and push again.
Thanks @sphuber , just having a look now again; the two methods `initialise` and `create_profile` are class methods. So moving the validation to constructor results in not being triggered during...
Thank a lot, @sphuber It all makes sense now, I updated accordingly. Cheers
Hi @sphuber , thanks a lot for being available for this PR. Yes, this is the same error as I encountered when I applied your changes locally. I think you...
Cheers, will do that