Julian Geiger
Julian Geiger
Also, to provide further context, the switch from psycopg2 to psycopg3 was done in PR #6362 (original issue is #5864), and was included first in v2.7.0, so this is not...
I just started looking into this issue, and for me the command `verdi profile dump --groups mygroup` command actually works fine: ```shell ❯ verdi profile dump --groups mygroup Warning: This...
Cheers, thanks for the great review, @agoscinski! I have implemented your proposed changes.
Closing this in favor of #6983. Please continue development there.
Hi @Muhammad-Rebaal, feel free to give it a shot and make a PR to solve this issue. To construct a test that reproduces the erroneous behavior, you can construct a...
@edan-bainglass, unfortunately, it is 🥲 https://github.com/aiidateam/aiida-core/blob/5e4da5b4dfd00b6ae270e020b8446887b1754230/src/aiida/storage/psql_dos/models/group.py#L41-L58 Also for the `DbLog` and `DbSetting` tables, btw. So, updating it to `ctime` everywhere would require a database migration :/ I get the original...
Thanks for the review, @danielhollas, all very good points! I applied your suggestions and will add you to the squash-merge message 🫶 > Ooof, good catch. I was incidentally recently...
> Hmm, but I don't think that the bug fixed in this PR should caused the crash though since the code as written should just skip over the duplicate job...
Adding some future architectural improvements here for when I revisit the implementation ## High Priority ### 1. Path Management Complexity **Problem:** `DumpPaths` class violates Single Responsibility Principle by handling path...
Closing in favor of meta-issue #6816.