David L. Dotson
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...
Blocking #292.
@ianmkenney can we characterize these failure further? Do we know why we see test suite failures when we set neo4j > 5.18?
Originally raised by @JenkeScheen here: https://github.com/asapdiscovery/asapdiscovery/issues/1196
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!