Julian Geiger
Julian Geiger
Alright, this should be ready for merge. I actually changed the option from `--sort-outputs` to `--unsorted`, with the default as false, so that people don't get surprised by the behavior...
As always, thanks for your help and assistance @sphuber, very much appreciated :pray:
Same here. Was very excited to find the extension, but this is really bothersome :/
> This sound like `verdi presto` should set the `use_login_shell` configuration since it is a required argument in the regular `verdi computer configure`. Is it okay if I change it...
Alright, cool! Should be done by #6403.
Note to self. There exists a similar fixture (factory) in aiida-qe: https://github.com/aiidateam/aiida-quantumespresso/blob/c4d7c190456deca0c26e4fc36304e973fe14a2c2/tests/conftest.py#L208 Also pinging @edan-bainglass, as @unkcpz mentioned you were also looking for this in the past. I'll clean up...
Hi @khsrali, that's a good idea. Initially, I didn't use the `generate_calcjob_node` fixture, because it is also function-scoped. Though, as of now, I just accept that and disregard fixture scopes,...
> Looks great. Please add two trivial dogfooding tests that uses these two fixtures. Thanks for looking into this, @unkcpz! A shame you did so right while I was updating...
> @GeigerJ2 have you considered aligning this fixtures with the already existing fixture: Thanks for looking into this, @khsrali. What exactly do you mean with _aligning_ in that case? This...
@unkcpz and @khsrali, I added a dogfood and a benchmark test. From the benchmark, one sees that constructing the node artificially indeed saves quite a bit of overhead: ``` ❯...