-
-
Notifications
You must be signed in to change notification settings - Fork 312
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
Geo distance transformer #688
Comments
Hey @VascoSch92 Yes, I also created distances from coordinates in my work as well. This is a functionality I've been wanting to add to the library for a long time. It'd be great if you could give it a go and and add that transformer. How would the transformer work? What I had in mind is that the transformer takes 2 lat and 2 lon variables, that is 4 input features, and returns the distance. That's how my data looked like in the past. Is this what you do as well? I am happy to hear of alternatives. Thank you! |
I have a first version of the distance transformer that I will like to share. What it the etiquette for branch names? |
Hey @solegalli , Thanks |
Hey @VascoSch92 Thank you so much for the engagement, and super apologies for the delayed response. Before taking new PRs I wanted to fix all tests that were failing due to the new releases of pandas, sklearn and scipy. That is now done. We work with PRs. So if you fork this repo (make sure you have the latest version of main), and then create a branch where you add the functionaity, you could make a PR from there. More info in how to contribute here: https://feature-engine.trainindata.com/en/latest/contribute/contribute_code.html |
Hei,
For a project, I created a transformer that given the coordinates of two points on the earth calculates the distance between them. It would be interesting to have in here.
If you are interested I could open a PR with a first draft to discuss it.
Let me know.
The text was updated successfully, but these errors were encountered: