Open otcathatsya opened 1 month ago
Right now the regular histentry operates on double, which is only needed for precise spike times. Since the majority of simulations use on-grid simulation steps, it may make sense to change the default to size_t and separate the precise times implementation (potentially through the Time class?). https://github.com/nest/nest-simulator/blob/b32a3a35f909b81db13073e6cf861491069f39e7/nestkernel/histentry.h#L40
double,
size_t
This would also simplify optimizations relying on grid-locked time steps, e.g. https://github.com/nest/nest-simulator/pull/3247.
See also https://github.com/nest/nest-simulator/pull/2035
clinssen
commented
1 month ago clinssen
commented
1 month ago
Right now the regular histentry operates on
double,which is only needed for precise spike times. Since the majority of simulations use on-grid simulation steps, it may make sense to change the default tosize_tand separate the precise times implementation (potentially through the Time class?). https://github.com/nest/nest-simulator/blob/b32a3a35f909b81db13073e6cf861491069f39e7/nestkernel/histentry.h#L40This would also simplify optimizations relying on grid-locked time steps, e.g. https://github.com/nest/nest-simulator/pull/3247.