Oops, missed this review. Adding some extra detail.
Very good drive - basically a Crucial MX500 clone. The MX500 often comes with 96L flash now (which is also denser, but has twice the planes). Team's coding here is clearly from Intel/Micron with the "01T" meaning 1Tb packages (x4 = 512GiB). This is in contrast to the similar L5 Lite 3D and drives like it - SU800, X3, S280, etc - which use inferior flash. Much like the L5 Lite 3D, it has static SLC - ~6GB at this capacity - which makes for uniform performance (can see this in HD Tune); the MX500 has a significantly larger, dynamic cache that is slower outside SLC at this capacity. Warranty is of course lackluster, but if you want a top-tier SATA drive this is among them in terms of performance.
The firmware revision matches typical SM2258 encoding which usually refers to the type of flash it's using. For example you common see Qx6/7xxA in the cheaper drives using second-tier Samsung TLC, Hynix's 72L TLC (which isn't great), inferior BiCS (SanDisk/Kioxia), older Intel/Micron 32L TLC, etc. Older revisions were for the 32L/384Gb IMFT TLC. In any case, R/S is for the newer and better flash products, e.g. B16A/B17A which is 64L Intel/Micron TLC (B17A is double the density but also double the planes, so same interleaving).
My only advice for reviewing is that FIO is a strong option for benchmarking, whether on Linux or Windows, although ultimately you would require Linux for full utility. I've posted on Reddit about using FIO and also making a dedicated Linux benchmarking USB drive, I haven't posted about getting these tools working on Windows - it is possible, I have it working in Visual Studio (direct) or Cygwin (indirect), but it's fairly limited and a pain to work with in contrast to nvme-cli for example.