lobe / iOS-bootstrap

Bootstrap your Lobe machine learning model with our iOS starter project.
https://lobe.ai
MIT License
99 stars 18 forks source link

App polish and new README. #24

Closed RamonGilabert closed 3 years ago

RamonGilabert commented 3 years ago

Hey,

I added app polish to align it with the original design—not changing the logic other than adding 3 predictions instead of one. I refactored the spacings and the tabs (2 tabs vs 4 spaces to reduce the need to scroll sideways). And changed the model file to be one that's more humane. See the results here:

https://user-images.githubusercontent.com/6138120/110975718-646e9580-8360-11eb-927a-4799da91edb5.mov

Considerations on the app:

The app is working perfectly on all breakpoints and test devices. The only considerations are:

  1. To reduce setup needed, I deleted the entitlements to access iCloud and Push notifications for now—we can add those later when they have a better purpose.
  2. I've simplified the provisioning profile needed to run—everyone should be able to run this now.
  3. The app doesn't work on the iPad anymore. iOS for now, we can add iPadOS later when we see need. Users can toggle it on, but it's at their own expense.

Things needed:

  1. There is an animation at the beginning that's broken, but I can't fix it.
  2. There should be an animation between the labels.
  3. The logic of adding 3 predictions is mostly fine, but it would be great to take a look and iron it out a bit more.

On the README, I simplified the language, though @billba will take a look, I added the header image, I removed +80 Mb of unneeded files and I've added consistency across in the file structure.

header