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

Upgrade Cyclomedia API to latest version 18.4 #971

Open
croswell81 opened this issue Feb 5, 2020 · 3 comments
Open

Upgrade Cyclomedia API to latest version 18.4 #971

croswell81 opened this issue Feb 5, 2020 · 3 comments
Labels
Greenskeeper To be handled by rotating, on-call Greenskeeper role.

Comments

@croswell81
Copy link

croswell81 commented Feb 5, 2020

Describe the bug
Cyclomedia imagery for the latest capture displays with pixilation around the seams of the imagery

To Reproduce
Steps to reproduce the behavior:

  1. Search for a property in Zola
  2. Scroll down and click on the Cyclomedia Street View link to launch the Cyclomedia window
  3. pan around the image to see pixilation

Expected behavior
Image should be seamless with no pixilation

Screenshots
image

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser: Chrome
  • Version 78.0.3904.97

Additional context
Spoke with Cyclomedia and they said upgrading to latest version of API will fix the issue. This is from their website:
As of August 29, CycloMedia will use a new way to store Cycloramas. Street Smart and the Street Smart API from version 18.4 onwards are prepared for this update, so there will be no noticeable changes.

@allthesignals
Copy link
Collaborator

Thanks @croswell81, great catch!

Relatedly, we made a change to the lot view where the Cyclomedia viewer doesn't embed: https://zola.planning.nyc.gov/l/lot/1/406/24

Have you noticed any performance improvements or any comments from users?

@hannahkates hannahkates added the Greenskeeper To be handled by rotating, on-call Greenskeeper role. label Mar 17, 2020
@allthesignals
Copy link
Collaborator

The code for this view can be found here: https://github.com/NYCPlanning/labs-cyclomedia-service

@allthesignals
Copy link
Collaborator

@croswell81 the Cyclomedia view is handled by another repo linked above. If we can get the mentioned PR (NYCPlanning/labs-cyclomedia-service#2) merged, it will fix the pixelation issue:

17.2:
Screen Shot 2021-10-26 at 4 40 54 PM

18.4:
Screen Shot 2021-10-26 at 4 41 07 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Greenskeeper To be handled by rotating, on-call Greenskeeper role.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants