Add legend to feature density maps

[atmorling]

This needs some thought around the display values for the legend, the density column alone won't be very descriptive

[cisaacstern]

Based this blame

https://github.com/wildlife-dynamics/ecoscope-workflows/blame/4b0d970d0bc41691f703ca059049c3c4226ca1b9/examples/events/ecoscope-workflows-events-workflow/ecoscope_workflows_events_workflow/dags/jupytext.py#L436

which shows that line being added in #460, I assume perhaps your last recompile commit on #460 may have been against a local version of the spec that had some experimental inline defaults, i.e.

    legend:
       label_column: "density",
       color_column": "density_colormap"

which were compiled against in #460 but not themselves committed?

I am trying to figure out why those params are here

ecoscope-workflows/examples/events/ecoscope-workflows-events-workflow/ecoscope_workflows_events_workflow/dags/jupytext.py

Lines 429 to 436 in 4b0d970

	fd_map_layer = create_polygon_layer.partial(
	geodataframe=fd_colormap,
	layer_style={
	"fill_color_column": "density_colormap",
	"get_line_width": 0,
	"opacity": 0.4,
	},
	legend={"label_column": "density", "color_column": "density_colormap"},

but not here

ecoscope-workflows/examples/events/spec.yaml

Lines 147 to 155 in 4b0d970

	- name: Create map layer from Feature Density
	id: fd_map_layer
	task: create_polygon_layer
	partial:
	geodataframe: ${{ workflow.fd_colormap.return }}
	layer_style:
	fill_color_column: "density_colormap"
	get_line_width: 0
	opacity: 0.4

They are being dropped by the auto-recompiles in #452, and just want to make sure that's not a mistake. I think I'm convinced its not.

[cisaacstern]

Put another way, is the above a case of us not having this ci check #433 ?

[atmorling]

Yep, that's 100% what happened 😅