-
Notifications
You must be signed in to change notification settings - Fork 9
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
V2.1.1 -V3: (NEW FEATURE) AR Ruler Measurement (beta) for DBH #219
Comments
Working on this design! 2-3w timeline. |
This comment has been minimized.
This comment has been minimized.
Just emailed you back, Ari - I saw your comments and just gave you editor permission to be able to make more edits yourself. I've had some recent scheduling changes, so I'm not sure when I'd be able to come back around to this! |
Thanks @kailynnelson ! |
I looked into adding AR ruler functionality to the app. unfortunately I could not get the code to work given time limitations, but it was still a useful exercise and there are some insights I can share. Overall I think that adding it through Viro is probably the way to go. The tutorial I used is found here: https://docs.viromedia.com/docs/viro-platform-overview. I couldn't get it to work, but the tutorial is well-written and with more time to play around with it it could work. The only downside to Viro is that there seems to be no code example of an AR ruler. This means that we would have to use Viro's API to somehow create our own way to measure distance, which may be complicated (see viromedia/viro#807 and viromedia/viro#54) An alternative could be using ARCore (https://shibuiyusuke.medium.com/measuring-distance-with-arcore-6eb15bf38a8f) for android or ARKit for iOS (see https://www.thedroidsonroids.com/blog/how-to-create-a-measuring-app-with-arkit-in-ios-11 or https://virtualrealitypop.com/ios-11-tutorial-how-to-measure-objects-with-arkit-743d2ec78afc). |
I tried a lot of research to implement this feature. during development, the library or dependencies for measuring dbh of the tree may be changed, the main goal is to get the dbh value.
it requires some maths and AI technologies as well here is the GitHub repository link for depth map. |
Testing notes (Feb 24 2022 Android APK)
|
@christyheaton (thanks for testing - and thank you @andaniel2029 for this great new tool!) I'd add to Christy's #1 test note (the negative values note) that maybe the measuring process starts over again after the user closes the 'Calibration error! Please try again.' msg box. |
Testing notes: Works well with medium trees where the trunk is clearly visible! Some changes that would be helpful: Instead of having to click the "?" icon to see instructions of how to use the app, have the following text appear by default when you open the DBH calculator: "Fit the width of the tree trunk at breast height (4.5 ft) between the two lines, then hit the "check icon". (For multiple trunks, select the "?" at bottom of screen)." The, when user hits the "?" the message should show up with the instructions for multiple trunks (as a popup, perhaps?): "Multiple Trunks: For each individual trunk, fit the width of the trunk at breast height (4.5 ft) between the two lines. Select "Enter Another" to add the next trunk." After the user has taken the first measurement and clicked the check icon, have the text in the screen say: "Center the line at the bottom of the tree, then touch the "camera icon"." |
Now available for all users in release 63!! Great job team (@christyheaton and @andaniel2029 and @kailynnelson )! |
ToDO:
** note: to add the AR option it seems VERY likely that we might have to convert the expo project to straight react-native**
Once the above is done/tested:
This feature will be released in a future release with the note "beta" (on the DBH selection options).
Current examples to follow for the AR measure design screen (similar to our design, the user aligns the tree width to the 2 vertical lines):
https://www.arboreal.se/en/new-method-to-measure-a-tree/
Check this thread for possible react-native package options (viro is suggested)
https://www.reddit.com/r/reactnative/comments/dwkwvj/is_react_native_have_similar_package_like_ar_ruler/
viro:
https://github.com/viromedia/viro
The text was updated successfully, but these errors were encountered: