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.

Sign up for GitHub

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

Jump to bottom

Issue #23260, #23267 reopen since the commit #24181 #25521

Closed
Kliffh opened this issue Feb 17, 2023 · 2 comments · Fixed by #25522
Closed

Issue #23260, #23267 reopen since the commit #24181 #25521

Kliffh opened this issue Feb 17, 2023 · 2 comments · Fixed by #25522
Labels
Regression

Comments

@Kliffh
Copy link

Kliffh commented Feb 17, 2023

Description

On ios14.x, the EXRLoader can not load hdri since the commit #24181. The func parseInt64 caused this.

Reproduction steps

1.using r147/r148 EXRLoader to load hdri
2.can not start the loading neither in safari nor chrome

Code

// code goes here

Live example

Screenshots

No response

Version

r148

Device

Mobile

Browser

Safari

OS

iOS
@LeviPesin
Copy link
Contributor

It seems indeed DataView.getBigInt64() is not supported in Safari 14 (but it should be supported in Chrome?). I can revert that change of parseInt64.

@mrdoob
Copy link
Owner

mrdoob commented Feb 17, 2023

@LeviPesin We should probably revert that change yes.

@LeviPesin LeviPesin mentioned this issue Feb 17, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Regression
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

EXRLoader: Fix parseInt64() regression. LeviPesin/three.js-1
4 participants
@mrdoob @Mugen87 @LeviPesin @Kliffh