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

Code Updated #15

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Code Updated #15

wants to merge 4 commits into from

Conversation

das-anubhav
Copy link

Fixed the on tap keyboard pop up issue
Fixed UI for iPhone X and above
Cleaned some code and removed the FPS anD Node count
Modified Accuracy as Confidence Percentage

@@ -191,7 +187,7 @@ class ViewController: UIViewController, ARSCNViewDelegate {
// Get Classifications
let classifications = observations[0...1] // top 2 results
.flatMap({ $0 as? VNClassificationObservation })
.map({ "\($0.identifier) \(String(format:"- %.2f", $0.confidence))" })
.map({ "\($0.identifier) \(String(format:"- %.2f", ($0.confidence)*100)) \("%")" })
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Earlier it showed only the confidence and the output used to be eg: 0.7632
Now it will show in percentage and the output comes as eg: 76.32 %

@@ -30,8 +30,7 @@ class ViewController: UIViewController, ARSCNViewDelegate {
// Set the view's delegate
sceneView.delegate = self

// Show statistics such as fps and timing information
sceneView.showsStatistics = true

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cleared the statistics for fps as it was effecting the user experience

super.didReceiveMemoryWarning()
// Release any cached data, images, etc that aren't in use.
}

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cleared as most of those problems were back in the iPhone 3GS days when memory was more limited.

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