David L. Dotson

Results 237 comments of David L. Dotson

We should wait to merge any solution to #291 until after we have resolved this, otherwise it's possible that our changes pass our tests for 5.18 but don't actually work...

@ianmkenney can we characterize these failure further? Do we know why we see test suite failures when we set neo4j > 5.18?

Thanks for raising this @LilDojd! Really appreciate your help with this. :pray: Will review your PR #306 as part of our next sprint cycle.

@LilDojd thanks again for doing this! I simplified the implementation to avoid unnecessary state in the `BcryptPasswordHandler`, along with comments on the rationale for various bits. Let me know what...

Excellent, thank you for that sanity check @LilDojd, as well as the NULL check! I'll merge shortly once tests pass!

@mnmelo the `Topology` object only holds the information; it doesn't manage the `AtomGroup`, `ResidueGroup`, and `SegmentGroup` definitions. The `Universe` is ultimately what does this, so it make sense for the...

Excellent, thanks for this @ijpulidos!