Alexander Goscinski
Alexander Goscinski
so annoying, on the c++ side we use size_t, so one has to do the check on the python side. I really think size_t is always a bad choice for...
Hello, before adding librascal to pypi, it makes sense to me that we have a stable model json format. The PRs #305 and #376 add significant changes to it. So...
After a conversation with @max-veit I realized that the only thing, which will not be backwards compatible due to #376, are cpp representations. I don't think any python user tries...
Don't you think `expansion_by_species_method` is the better variable name to keep? Open for any other naming, `species_list` is just a bad name for variable which can also be a string
find it confusing when variable name contains a data type and also accepts other data types. I would go with something as `species_storage` but its not so important, its your...
For this problem introducing the solution for #213 a proper error message is now thrown. > Another issue that just came up when I tried a gradient test of `small_molecule.json`...
put this issue into the developer doc
> Very nice work, thanks @agoscinski! For me, the naming is fine. I just dogfooded it a bit by running `verdi computer export --help`, and exporting and re-importing my `daint-gpu`...
I squashed all changes from last review into the first commit, so all new commits are from the second round. The incomplete config @GeigerJ2 mentioned when using verdi presto is...