Marnik Bercx

Results 304 comments of Marnik Bercx
trafficstars

> If and only if the target directory already exists right? Otherwise it just copies as is. Not sure if that's true, see my example near the end of https://github.com/aiidateam/aiida-core/pull/6348#discussion_r1594585968

One note while testing this. `verdi status` seems to still refer to `main_0001`: ``` ❯ verdi status ✔ version: AiiDA v2.5.1.post0 ✔ config: /Users/mbercx/project/core/.aiida ✔ profile: presto ✘ storage: Storage...

A second comment was related to the fact that I noticed migrations were still stored in the `psql_dos` directory. This made me wonder: can we migrate `sqlite_dos` storages? I tried...

> However, I think we should implement in this case some error handlers to just split the calculation in chunks and then merge it back (or even know already that...

One more note: maybe we should also look at the protocol for the NSCF part. I mean we just arbitrarily set `kpoints_distance` to a _very_ dense `0.05` for `precision`. Perhaps...

Thanks @bastonero! But I'm afraid at first glance I'm not too happy with the solution provided by ChatGPT. ^^ Having to wrap every `get_builder_from_protocol()` method seems tedious, it should just...

> Anyway, as much as I don't like the naming and as much as I dislike having to look at names I don't like, I doubt whether the cost for...

Just adding this here, since the issue is similar. With `aiida-core==2.4.1` I'm running into the following: ``` ❯ verdi daemon restart --reset --timeout 20 Profile: dev Stopping the daemon... OK...

Tests pass, but maybe we should add some more to properly test the functionality with and without the `energy_range_vs_fermi` input? Also: maybe we should move the long module docstring into...

@sphuber she's almost ready, just a bit more cleaning. I would also: 1. Remove the `v.` for the code versions. 2. Remote the `wall_time`. It's a useless string and there...