-
Notifications
You must be signed in to change notification settings - Fork 429
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
GEOMESA-3238 Support for Non-Normalized Heatmaps #2904
base: main
Are you sure you want to change the base?
GEOMESA-3238 Support for Non-Normalized Heatmaps #2904
Conversation
mkearns-ccri
commented
Oct 18, 2022
- Adds normalization boolean to DensityProcess
- Updates density process documentation to reflect the change
- Adds scala implementation of the HeatmapSurface class
* Adds normalization boolean to DensityProcess * Updates density process documentation to reflect the change * Adds scala implementation of the HeatmapSurface class
|
||
import org.locationtech.jts.geom.Envelope | ||
|
||
class HeatmapSurface(kernelRadius: Int, |
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.
@elahrvivaz I basically just ripped this from geotools and added a ccri copyright header - not sure if this is okay or not. Unfortunately I had to copy the whole class because java doesn't allow overriding private methods. Thoughts?
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.
i was wondering about that too and i'm not sure of the answer - would it be possible to use the existing geotools classes, but track the max value when adding points, then de-normalize it afterwards? then we wouldn't have to worry about it
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.
I confirmed with @jnh5y that we're not allowed to copy and re-license geotools as it's LGPL, so we'll need to come up with a different solution here.
|
||
import org.locationtech.jts.geom.Envelope | ||
|
||
class GridTransform(env: Envelope, xSize: Int, ySize: Int, isClamped: Boolean = true) { |
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.
same here
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.
marking this a needing changes due to the license of the geotools files we copied.