Skip to content
This repository has been archived by the owner on Oct 16, 2023. It is now read-only.

Ensure gafferpy works correctly with HLLP #1036

Closed
t92549 opened this issue Nov 15, 2022 · 1 comment · Fixed by #1051
Closed

Ensure gafferpy works correctly with HLLP #1036

t92549 opened this issue Nov 15, 2022 · 1 comment · Fixed by #1051

Comments

@t92549
Copy link
Contributor

t92549 commented Nov 15, 2022

Ensure the changes added in gchq/Gaffer#2749 work with gafferpy
Namely, make sure the HyperLogLogPlus class in Python can be JSON deserialised

@t92549 t92549 added this to the v2.0.0-alpha-0.4 milestone Nov 15, 2022
@t92549 t92549 self-assigned this Nov 15, 2022
t92549 added a commit that referenced this issue Mar 6, 2023
* Added more types and examples for hllp

* autopep8 action fixes (#1052)

* Types improvements

* Added return types to gaffer_types.py

* autopep8 action fixes (#1053)
---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: t92549 <t92549@users.noreply.github.com>
@t92549 t92549 linked a pull request Mar 6, 2023 that will close this issue
@t92549
Copy link
Contributor Author

t92549 commented Mar 6, 2023

Closed by #1051

@t92549 t92549 closed this as completed Mar 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant