Ali Khosravi

Results 107 comments of Ali Khosravi

Hi @FARDEEN-785, If that helps you to grasp more understanding of the code base for future contributions, then sure, by all means go ahead!

Hi @Muhammad-Rebaal Check this file: `test_all_plugins.py` some tests although being in place, they don't really cover the pitfalls. You can think through and imagine all scenarios for each function, and...

@Muhammad-Rebaal doing that, you can skip any test directly related to `get`, `put`, and `copy` methods. I already have fix patches for them, which I plan to commit in the...

Just talked to @agoscinski , We don't actually need to store the `uuid` actually. We could just use the traversals to reach there: ```python from aiida.orm import load_node, CalcJobNode, Node...

@agoscinski I applied your previous review applied, Please have a look on Monday, now it should be a quick review

I applied your review @agoscinski We can talk in person about QB

> We discussed that we need a migration for this as we do not want to have two datatypes in the database that basically do the same thing. Introducing the...

I close this for now. The implementation of this commit might get used in a more broad PR that deprecates all `RemoteStash*Data` by removing abstractification in `RemoteStashData`

Because we don't want to do it manually, and because it should not happen. :-)

Hi @crs7617, I think also @ayushjariyal showed some interest working on this. I don't know if he's still working on it or not.