Jonathan Chico
Jonathan Chico
Looking at this away, it seems to be that the issue comes from the fact that the `simplejson` dependency was removed and the base `json` package is used instead. This...
@sphuber thanks for the answer. I think if that is what the standard requires that is what we should do. It is of course not ideal that we cannot identify...
I have made a small PR on this, basically it is the same function that I described before but I also included the check `np.isinf` so those are also transformed...
@sphuber, @chrisjsewell Do you agree with this PR? Or should something else be done before merging?
Hi @sphuber! > I'm happy with the concept of changing `inf` and `nan` to `None`. We should probably just add a test and maybe update the documentation of the REST...
@sphuber done, test and note in docs added. If something else is needed just let me know.
Hi @sphuber , funnily enough it seems to be that there were some issues with unavailable resources (in other areas of the code), for example one of the errors that...
So removing the `numpy.ndarray` type hinting and fixing the broken links worked. I will squash the last commits into a single one to have a cleaner history.
One way in which I solved this in the `UppASD` code was by creating two types of geometries, a "host" geometry (which would be equivalent to the used now) and...
> What I suggested in the Slack discussion, as a purely hacky work-around, is to compute the hash of the node with the new inputs and then override the _aiida_hash...