Merge branch 'main' into add_event_handlers_to_drawn_layers_in_draw_plugin

Author: hansthen

.github/workflows/deploy-docs.yml

@@ -25,7 +25,7 @@ jobs:
         ref: main
 
     - name: Setup Micromamba env
-      uses: mamba-org/setup-micromamba@v1
+      uses: mamba-org/setup-micromamba@v2
       with:
         environment-name: TEST
         create-args: >-

.github/workflows/pypi.yml

@@ -46,7 +46,7 @@ jobs:
 
     - name: Publish a Python distribution to PyPI
       if: success() && github.event_name == 'release'
-      uses: pypa/[email protected].2
+      uses: pypa/[email protected].3
       with:
         user: __token__
         password: ${{ secrets.PYPI_PASSWORD }}

.github/workflows/test_code.yml

@@ -27,7 +27,7 @@ jobs:
     - uses: actions/checkout@v4
 
     - name: Setup Micromamba env
-      uses: mamba-org/setup-micromamba@v1
+      uses: mamba-org/setup-micromamba@v2
       with:
         environment-name: TEST
         create-args: >-

.github/workflows/test_latest_branca.yml

@@ -14,7 +14,7 @@ jobs:
     - uses: actions/checkout@v4
 
     - name: Setup Micromamba env
-      uses: mamba-org/setup-micromamba@v1
+      uses: mamba-org/setup-micromamba@v2
       with:
         environment-name: TEST
         create-args: >-

.github/workflows/test_mypy.yml

@@ -14,7 +14,7 @@ jobs:
     - uses: actions/checkout@v4
 
     - name: Setup Micromamba env
-      uses: mamba-org/setup-micromamba@v1
+      uses: mamba-org/setup-micromamba@v2
       with:
         environment-name: TEST
         create-args: >-

.github/workflows/test_selenium.yml

@@ -18,7 +18,7 @@ jobs:
     - uses: actions/checkout@v4
 
     - name: Setup Micromamba env
-      uses: mamba-org/setup-micromamba@v1
+      uses: mamba-org/setup-micromamba@v2
       with:
         environment-name: TEST
         create-args: >-

.pre-commit-config.yaml

@@ -2,7 +2,7 @@ exclude: examples/data/|.*\.css|.*\.json|.*\.geojson|.*\.html
 
 repos:
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: v4.6.0
+  rev: v5.0.0
   hooks:
     - id: trailing-whitespace
     - id: check-ast
@@ -15,7 +15,7 @@ repos:
       files: requirements-dev.txt
 
 - repo: https://github.com/astral-sh/ruff-pre-commit
-  rev: v0.6.3
+  rev: v0.6.9
   hooks:
     - id: ruff
 

folium/plugins/draw.py

@@ -12,6 +12,9 @@ class Draw(JSCSSMixin, MacroElement):
     ----------
     export : bool, default False
         Add a small button that exports the drawn shapes as a geojson file.
+    feature_group : FeatureGroup, optional
+        The FeatureGroup object that will hold the editable figures. This can
+        be used to initialize the Draw plugin with predefined Layer objects.
     filename : string, default 'data.geojson'
         Name of geojson file
     position : {'topleft', 'toprigth', 'bottomleft', 'bottomright'}
@@ -63,10 +66,17 @@ class Draw(JSCSSMixin, MacroElement):
               draw: {{ this.draw_options|tojson }},
               edit: {{ this.edit_options|tojson }},
             }
-            // FeatureGroup is to store editable layers.
-            var drawnItems_{{ this.get_name() }} = new L.featureGroup().addTo(
-                {{ this._parent.get_name() }}
-            );
+            {%- if this.feature_group  %}
+                var drawnItems_{{ this.get_name() }} =
+                    {{ this.feature_group.get_name() }};
+            {%- else %}
+                // FeatureGroup is to store editable layers.
+                var drawnItems_{{ this.get_name() }} =
+                    new L.featureGroup().addTo(
+                        {{ this._parent.get_name() }}
+                    );
+            {%- endif %}
+
             options.edit.featureGroup = drawnItems_{{ this.get_name() }};
             var {{ this.get_name() }} = new L.Control.Draw(
                 options
@@ -89,7 +99,11 @@ class Draw(JSCSSMixin, MacroElement):
                 );
                 {%- endfor %}
                 drawnItems_{{ this.get_name() }}.addLayer(layer);
-             });
+            });
+            {{ this._parent.get_name() }}.on('draw:created', function(e) {
+                drawnItems_{{ this.get_name() }}.addLayer(e.layer);
+            });
+            
             {% if this.export %}
             document.getElementById('export').onclick = function(e) {
                 var data = drawnItems_{{ this.get_name() }}.toGeoJSON();
@@ -123,6 +137,7 @@ class Draw(JSCSSMixin, MacroElement):
     def __init__(
         self,
         export=False,
+        feature_group=None,
         filename="data.geojson",
         position="topleft",
         show_geometry_on_click=True,
@@ -133,6 +148,7 @@ def __init__(
         super().__init__()
         self._name = "DrawControl"
         self.export = export
+        self.feature_group = feature_group
         self.filename = filename
         self.position = position
         self.show_geometry_on_click = show_geometry_on_click

folium/plugins/heat_map_withtime.py

@@ -86,7 +86,7 @@ class HeatMapWithTime(JSCSSMixin, Layer):
                 speedStep: {{this.speed_step}},
                 styleNS: "{{this.style_NS}}",
                 timeSlider: {{this.time_slider}},
-                timeSliderDrapUpdate: {{this.time_slider_drap_update}},
+                timeSliderDragUpdate: {{this.time_slider_drag_update}},
                 timeSteps: {{this.index_steps}}
                 })
                 .addTo({{this._parent.get_name()}});
@@ -199,7 +199,7 @@ def __init__(
         self.time_slider = "true"
         self.play_button = "true"
         self.play_reverse_button = "true"
-        self.time_slider_drap_update = "false"
+        self.time_slider_drag_update = "false"
         self.style_NS = "leaflet-control-timecontrol"
 
     def render(self, **kwargs):

folium/plugins/time_slider_choropleth.py

@@ -19,6 +19,8 @@ class TimeSliderChoropleth(JSCSSMixin, Layer):
     styledict: dict
         A dictionary where the keys are the geojson feature ids and the values are
         dicts of `{time: style_options_dict}`
+    date_options: str, default "ddd MMM DD YYYY"
+        A format string to render the currently active time in the control.
     highlight: bool, default False
         Whether to show a visual effect on mouse hover and click.
     name : string, default None
@@ -44,6 +46,11 @@ class TimeSliderChoropleth(JSCSSMixin, Layer):
             let styledict = {{ this.styledict|tojson }};
             let current_timestamp = timestamps[{{ this.init_timestamp }}];
 
+            function formatDate(date) {
+               var newdate = new moment(date);
+               return newdate.format({{this.date_format|tojson}});
+            }
+
             let slider_body = d3.select("body").insert("div", "div.folium-map")
                 .attr("id", "slider_{{ this.get_name() }}");
             $("#slider_{{ this.get_name() }}").hide();
@@ -64,7 +71,7 @@ class TimeSliderChoropleth(JSCSSMixin, Layer):
                 .attr("step", "1")
                 .style('align', 'center');
 
-            let datestring = new Date(parseInt(current_timestamp)*1000).toDateString();
+            let datestring = formatDate(parseInt(current_timestamp)*1000);
             d3.select("#slider_{{ this.get_name() }} > output").text(datestring);
 
             let fill_map = function(){
@@ -84,7 +91,7 @@ class TimeSliderChoropleth(JSCSSMixin, Layer):
 
             d3.select("#slider_{{ this.get_name() }} > input").on("input", function() {
                 current_timestamp = timestamps[this.value];
-                var datestring = new Date(parseInt(current_timestamp)*1000).toDateString();
+                let datestring = formatDate(parseInt(current_timestamp)*1000);
                 d3.select("#slider_{{ this.get_name() }} > output").text(datestring);
                 fill_map();
             });
@@ -155,12 +162,19 @@ class TimeSliderChoropleth(JSCSSMixin, Layer):
         """
     )
 
-    default_js = [("d3v4", "https://d3js.org/d3.v4.min.js")]
+    default_js = [
+        ("d3v4", "https://d3js.org/d3.v4.min.js"),
+        (
+            "moment",
+            "https://cdnjs.cloudflare.com/ajax/libs/moment.js/2.18.1/moment.min.js",
+        ),
+    ]
 
     def __init__(
         self,
         data,
         styledict,
+        date_options: str = "ddd MMM DD YYYY",
         highlight: bool = False,
         name=None,
         overlay=True,
@@ -173,21 +187,21 @@ def __init__(
     ):
         super().__init__(name=name, overlay=overlay, control=control, show=show)
         self.data = GeoJson.process_data(GeoJson({}), data)
+        self.date_format = date_options
         self.highlight = highlight
 
         self.stroke_opacity = stroke_opacity
         self.stroke_width = stroke_width
         self.stroke_color = stroke_color
 
         if not isinstance(styledict, dict):
-            raise ValueError(
-                f"styledict must be a dictionary, got {styledict!r}"
-            )  # noqa
+            raise ValueError(f"styledict must be a dictionary, got {styledict!r}")
+
         for val in styledict.values():
             if not isinstance(val, dict):
                 raise ValueError(
                     f"Each item in styledict must be a dictionary, got {val!r}"
-                )  # noqa
+                )
 
         # Make set of timestamps.
         timestamps_set = set()