-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Knox enhancements #111
Knox enhancements #111
Conversation
Codecov Report
@@ Coverage Diff @@
## main #111 +/- ##
==========================================
+ Coverage 59.57% 66.28% +6.70%
==========================================
Files 10 12 +2
Lines 1405 1732 +327
Branches 246 0 -246
==========================================
+ Hits 837 1148 +311
- Misses 518 584 +66
+ Partials 50 0 -50
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is nice! Could we try to get this and #105 in and cut a release with sklearn 1.3.0 compatibility fixes we did here?
Yes. Perhaps during the sprint on 8-24? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
add dataframe constructor
rm errant decorator
dont overwrite gdf
fix explore nonsig
@martinfleis not to distract you from working on lib at the moment, but a quick thing in case you have an idea.. we cooked up a little explore method on the localknox class that works pretty nicely. One killer thing would be to have an onclick or mouseover function that highlights neighbors when you interact with focal. I think the current method has all the info necessary to get there, but not sure how to go about rigging that up with folium. Curious if you know how |
Not possible afaik. The styling function sees only a single item of the json and I didn't figure out a way of highlighting from B based on an interaction with a geom A. |
inference type parameter for hotspots
bummer. pretty sure that's dead simple in vanilla leaflet. I was trying to send back IDs of neighbors as an attribute of focal to highlight those vals in a different color or something thanks for the insight |
I am not so sure, folium is pretty close in API to leaflet. But I suppose it would need a bit of custom JS which is easy when you have a leaflet map, not so much with folium. Maybe bokeh or geoviews may offer solutions. |
Features
Todo
add testspolish user notebook (add plotting)