{"push_id":2863642468,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"6bef005b22a51f567e5a93b025150c0a4816ab7b","before":"b4fd06ee9b3a0281ecc52acc2862291c126a20f9","commits":[{"sha":"2f2d737522c9d942df7aa145f8de66a135958940","author":{"name":"Jake Lishman","email":"70542b5e9b80ff23666db8e4bd08ca71559248d8@gmail.com"},"message":"Change is_unitary to use decimals not tolerance\n\nThe function itself uses both, so its call structure doesn't really\nrequire one or the other.  Since it's always called from `FixedSystem`,\nthough, which has the precision stored as `decimals`, it makes sense for\nthe function to match.","distinct":true,"url":"https://api.github.com/repos/ImperialCQD/floq/commits/2f2d737522c9d942df7aa145f8de66a135958940"},{"sha":"6bef005b22a51f567e5a93b025150c0a4816ab7b","author":{"name":"Jake Lishman","email":"70542b5e9b80ff23666db8e4bd08ca71559248d8@gmail.com"},"message":"Tidy up FixedSystem\n\nThis changes the names of all the functions and internals of\n`FixedSystem` to match the new names in `floq.System`, and removes a lot\nof additional functions like `_compute_u()`, which is now just baked\ninto the underlying `__compute_eigensystem_and_u()`.\n\nThe separated functions were likely there only to facilitate the\nautomated testing of the caching.  I should come up with a way to have\nthe best of both worlds - having a lot of additional code just creates\nmore technical debt and should be avoided, but the concept of testing\nall the small details underneath is definitely desirable as well.  Here\nI opted to remove the additional functions because the parts which\nimplement the cache are very straightforward (`if self.x is not None`).\n\nIn the future, I will hopefully be removing a lot of this code anyway to\nmake way for a smarter cache, where only the eigensystem is stored so\nmultiple times can be evaluated: see #1.","distinct":true,"url":"https://api.github.com/repos/ImperialCQD/floq/commits/6bef005b22a51f567e5a93b025150c0a4816ab7b"}]}
{"actor":{"display_login":"jakelishman"}}