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

3DTiles spamming console.log with thousands of messages. #4313

Closed
mramato opened this issue Sep 12, 2016 · 2 comments
Closed

3DTiles spamming console.log with thousands of messages. #4313

mramato opened this issue Sep 12, 2016 · 2 comments

Comments

@mramato
Copy link
Contributor

mramato commented Sep 12, 2016

#4228 added a message about using the old 3DTiles b3dm format, but it's using console.log and spamming the output with thousands of messages when working with most existing 3DTiles sets. It should use oneTimeWarning or depracationWarning instead.

@pjcozzi
Copy link
Contributor

pjcozzi commented Sep 12, 2016

Agreed, this is commented out in the 3d-tiles-temp branch, but do not use that branch right now as it is experimental.

@lilleyse
Copy link
Contributor

lilleyse commented Sep 21, 2016

@JudyWeng let's do this one next. Use a deprecationWarning, for example like 72c7f4a#diff-5616be61ebe4c0b247de93dde619a130L295

For testing you can follow the same steps as here: #3892 (comment). You can also use the same tileset for testing since it is in the legacy format.

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

No branches or pull requests

3 participants