Skip to content
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

Bug: "Street View" button links to wrong address #981

Open
nlebovits opened this issue Oct 27, 2024 · 7 comments
Open

Bug: "Street View" button links to wrong address #981

nlebovits opened this issue Oct 27, 2024 · 7 comments
Assignees
Labels
bug Something isn't working frontend help wanted Extra attention is needed

Comments

@nlebovits
Copy link
Collaborator

Describe the bug
The "street view" button for 4307 N 16th St. links to 1535 W. Bristol St. This may be due to the properties being close to each other--unclear. Does not seem to happen for all properties, so it might be something weird specific to this property. Needs investigation.

Screenshots
image

Desktop (please complete the following information):
Running Firefox 131.0.3 on Ubuntu 24.

@nlebovits nlebovits added bug Something isn't working help wanted Extra attention is needed frontend labels Oct 27, 2024
Copy link

This issue has been marked as stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale label Nov 27, 2024
@rkillion
Copy link

Hi @nlebovits ! I attended the Hackathon night on Tuesday and spoke with @CodeWritingCow about contributing to this project. This looks like a good challenge I'd be happy to take on though I'm not sure if it is still an issue. If so, could you assign it to me? If not, could you recommend a different issue? I have pretty broad experience full stack so I'm happy to try anything you have.

@CodeWritingCow
Copy link
Collaborator

@rkillion assigned! Thanks for tackling this.

@rkillion
Copy link

@CodeWritingCow @nlebovits Sorry it took me a while to get going on this! But I was able to get installed and start taking a look this week, so just wanted to give you an update:

I was able to confirm that incorrect latitude and longitude are being passed to the StreetView component for this address.
Passing {lat: 40.01906368194187, lon: -75.15209555625916, ...} when it should be {lat: 40.0191167, lon: -75.1524003, ...} according to google maps. When the correct coordinates are passed manually to the StreetView component, it lands in the right location.

So I just need to work backwards from there to figure out where the incorrect coordinates are coming from. I should have some more time to work on it this week so hopefully I'll have another update soon!

@CodeWritingCow
Copy link
Collaborator

@rkillion thanks for the update! Please keep us posted.

@CodeWritingCow CodeWritingCow moved this from To Do to In Development in Clean & Green Philly Feb 1, 2025
@rkillion
Copy link

rkillion commented Feb 7, 2025

@CodeWritingCow @nlebovits , hope you are well! I was able to identify the issue and a possible fix for this. However, the fix I have requires using the Geocode API to get a more accurate streetview point, and you may have a better solution based on data already available in the database. Here's a short video showing the issue and solution: https://www.loom.com/share/c73041327f8e47979b69e5c9b8e20576?sid=3bbefb8e-f587-457c-bbb2-b347cfd85669

Check out the video when you get the chance and if that solution seems ok, let me know and I can move forward with it!

@CodeWritingCow
Copy link
Collaborator

@rkillion thank you for looking into this and explaining your findings in the loom video!

If I understand your video correctly, then it'd be more accurate to use Geocode API's driving/navigation coordinates. Does the Geocode API have a rate limit?

This project does have a Google API key being used for the maps streetview API. That API is getting called by our data pipeline (see data/src/streetview.py).

We are getting each vacant property's location from our data pipeline. I'm not sure if the pipeline is saving each property location as an address or as coordinates. @nlebovits would you happen to know?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend help wanted Extra attention is needed
Projects
Status: In Development
Development

No branches or pull requests

3 participants