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

bump niimath version and enable some safari on tablet and desktop #6

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hanayik
Copy link
Member

@hanayik hanayik commented Dec 23, 2024

This PR bumps niimath to the latest version and enables Safari as an allowed browser if running on desktop or iPadOS.

I have tested this on macOS 15.2, iPadOS 18.2, and iOS 18.2. The iOS device is correctly not allowed to run the models, but desktop and iPads are. Tested on macOS laptop with M3 Max, and iPad Pro with M4 chip. Admittedly, these devices are all "pro-level" and I have not tested with lower end mobile devices from the Apple ecosystem.

I did try to run the models on my iPhone 15 pro max and even the lightweight three class segmentation model fails. So These devices should be disallowed.

Users are now alerted if their device is likely to not work. Previously, this was a silent check and we never actually told users that their device probably won't work.

@hanayik hanayik requested a review from neurolabusc December 23, 2024 10:52
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 this pull request may close these issues.

1 participant