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

Layout benchmark broken (again) #1709

Closed
zhangyiatmicrosoft opened this issue Oct 6, 2022 · 5 comments · Fixed by #1711
Closed

Layout benchmark broken (again) #1709

zhangyiatmicrosoft opened this issue Oct 6, 2022 · 5 comments · Fixed by #1711

Comments

@zhangyiatmicrosoft
Copy link
Contributor

Very similar to previous two issues reported:
#1293
#1254

  1. npm run ci
  2. npm run start-bench
  3. open web browser at http://localhost:9966/bench/versions?#Layout
  4. console will show
benchmarks_generated.js:33 Uncaught TypeError: this.map.setTerrain is not a function at Jt._load (benchmarks_generated.js:33:102708) at benchmarks_generated.js:33:101895

The said script is https://maplibre.org/maplibre-gl-js/benchmarks/main/benchmarks_generated.js
NOT coming from my local box.

@HarelM
Copy link
Collaborator

HarelM commented Oct 6, 2022

Seems like this is related to a recent change with 3D terrain and the fact that the bench is using a map stub and that it needs to run manually. See #982 for more info.
I'll see if I can create a PR that fixes this... :-(

@HarelM
Copy link
Collaborator

HarelM commented Oct 6, 2022

Feel free to review my PR :-)

@HarelM
Copy link
Collaborator

HarelM commented Oct 6, 2022

@zhangyiatmicrosoft it would be great if you could make sure this is now fixed after the following actions is completed:
https://github.com/maplibre/maplibre-gl-js/actions/runs/3199696952
Thanks!!

@zhangyiatmicrosoft
Copy link
Contributor Author

Confirm that this is working now. Thanks again for the quick fix!

@HarelM
Copy link
Collaborator

HarelM commented Oct 6, 2022

No problem. Thanks for reporting this!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants