You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The constructor of MULTIBATH is not very convenient to use and requires initialization of many variables. Some of these variables are often identical (like COMBATH and IRBATH) or can be derived (like DOFSET):
How about we take an approach as taken here? For example, T and TAU can be a list or floats (as they are often the same for several baths). Also, I like the dictionary approach for last_atoms_bath.
@pultar does this need to be in release3? The way I see it this is not a major restructuring, but rather a addition we can add later if actually needed
The constructor of MULTIBATH is not very convenient to use and requires initialization of many variables. Some of these variables are often identical (like COMBATH and IRBATH) or can be derived (like DOFSET):
Initialization happens like that:
How about we take an approach as taken here? For example,
T
andTAU
can be a list or floats (as they are often the same for several baths). Also, I like the dictionary approach forlast_atoms_bath
.Ideally, we would implement this in a way that does not break things.
The text was updated successfully, but these errors were encountered: