Changes between Initial Version and Version 1 of Ticket #28365, comment 6


Ignore:
Timestamp:
08/23/19 01:40:26 (2 years ago)
Author:
slelievre
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #28365, comment 6

    initial v1  
    99
    1010Yes the use of `time()` was only a (bad!) attempt to guarantee uniqueness. Combining it with `getrandbits(128)` was naively thought as a security in case two successive calls of `time()` occur with a time separation too small to be resolved.
    11 The use of the monotically increasing integer as in you implemented in the above commit seems a much more robust solution.
     11The use of the monotonically increasing integer as in you implemented in the above commit seems a much more robust solution.
    1212
    1313In the long term, I think we should get rid of the `UniqueRepresentation` for manifolds, except for those that do have a mathematically meaningful unique representation, like `EuclideanSpace`. Having manifolds inherit from `UniqueRepresentation` was a hack to make pickling work easily, which is required for parallelized computations via `multiprocessing`.