Flowmachine pareto

[BhavinPanch]

Closes #1012

I have:

• Formatted any Python files with black
• Brought the branch up to date with master
• Added any relevant Github labels
• Added tests for any new additions
• Added or updated any relevant documentation
• Added an Architectural Decision Record (ADR), if appropriate
• Added an MPLv2 License Header if appropriate
• Updated the Changelog

Expose Pareto interactions via FlowAPI's joined spatial aggregate endpoint.

[maxalbert]

Can we please add a validate parameter here to ensure that proportion is a value (strictly) between 0.0 and 1.0?

[maxalbert]

Looks good. 👍

A couple of minor inline comments, then this is ready to merge (after resolving merge conflicts due to a previous PR having been merged already).

[codecov]

Codecov Report

Merging #1044 into master will decrease coverage by 1.21%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #1044      +/-   ##
==========================================
- Coverage   91.23%   90.02%   -1.22%     
==========================================
  Files         141       16     -125     
  Lines        6786     1393    -5393     
  Branches      662       57     -605     
==========================================
- Hits         6191     1254    -4937     
+ Misses        483      131     -352     
+ Partials      112        8     -104

Flag	Coverage Δ
#flowapi_unit_tests	82.4% <ø> (ø)	⬆️
#flowauth_unit_tests	94.86% <ø> (ø)	⬆️
#flowclient_unit_tests	80.82% <50%> (-0.29%)	⬇️
#flowetl_unit_tests	?
#flowkit_jwt_generator_unit_tests	100% <ø> (ø)	⬆️
#flowmachine_unit_tests	?

Impacted Files	Coverage Δ
flowclient/flowclient/client.py	80.82% <50%> (-0.29%)	⬇️
...achine/core/server/query_schemas/daily_location.py
...e/flowmachine/features/raster/raster_statistics.py
flowetl/etl/etl/etl_utils.py
flowmachine/flowmachine/core/custom_query.py
flowmachine/flowmachine/core/server/server.py
...hine/features/location/unique_subscriber_counts.py
...ne/core/server/query_schemas/radius_of_gyration.py
...owmachine/features/subscriber/subscriber_degree.py
flowmachine/flowmachine/core/grid.py
... and 115 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2948654...900ae2a. Read the comment docs.

[codecov]

Codecov Report

Merging #1044 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1044      +/-   ##
==========================================
+ Coverage   94.03%   94.06%   +0.02%     
==========================================
  Files         144      145       +1     
  Lines        7030     7055      +25     
  Branches      694      694              
==========================================
+ Hits         6611     6636      +25     
  Misses        309      309              
  Partials      110      110

Flag	Coverage Δ
#flowapi_unit_tests	82.4% <ø> (ø)	⬆️
#flowauth_unit_tests	94.86% <ø> (ø)	⬆️
#flowclient_unit_tests	80.63% <50%> (-0.28%)	⬇️
#flowetl_unit_tests	100% <ø> (ø)	⬆️
#flowkit_jwt_generator_unit_tests	100% <ø> (ø)	⬆️
#flowmachine_unit_tests	91.04% <73.91%> (-0.08%)	⬇️
#integration_tests	64.18% <100%> (+0.73%)	⬆️

Impacted Files	Coverage Δ
...e/server/query_schemas/joined_spatial_aggregate.py	100% <100%> (ø)	⬆️
...e/core/server/query_schemas/pareto_interactions.py	100% <100%> (ø)
flowclient/flowclient/client.py	94.59% <100%> (+0.04%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d3f9d4d...94b5a7b. Read the comment docs.

[maxalbert]

Thanks for the changes! 👍

Nearly ready to merge, the only missing bit is to add a validate parameter in the proportions field of ParetoInteractionsSchema (see inline comment in query_schemas/pareto_interactions.py ).

[maxalbert]

Sorry, my bad. Github was pointing me to an old revision, but I saw you already changed this so this is good to go. Thanks!