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

unjustifiedZeroDBH #29

Open
ValentineHerr opened this issue Apr 13, 2023 · 5 comments
Open

unjustifiedZeroDBH #29

ValentineHerr opened this issue Apr 13, 2023 · 5 comments
Assignees

Comments

@ValentineHerr
Copy link
Member

ValentineHerr commented Apr 13, 2023

Census crew,

tag 10704 stemtag 2 is showing up with error unjustifiedZeroDBH bcause it has dbh = 0 but it is LI and has no codes.
In the note it says DBH 337.
Why was the DBH not entered int the appropriate spot?

@krystalbagnaschi
Copy link
Collaborator

We were getting an error that the dbh had to be between 0 and 200, and would not let us submit with a dbh of 337. We took a photo that I think Iris sent to Jess, as well as a screenshot on the iPad.

@jess-shue
Copy link
Contributor

@krystalbagnaschi I'll fix that - I had DBH limits set for cm; it was corrected on the tree table but not the stem table. Will it be OK to push that correction this morning, or should it wait?

@mitreds
Copy link
Collaborator

mitreds commented Apr 14, 2023 via email

@jess-shue
Copy link
Contributor

jess-shue commented Apr 14, 2023

@mitreds Should be fixed - I did reload the map on my iPad and was able to enter a DBH over 200 on a secondary stem.

ValentineHerr added a commit that referenced this issue May 9, 2023
@rmh24, @Iriskennedy, @CarolineTroy, @krystalbagnaschi
FYI, I redownloaded the data after you've fixed a lot of it.
- The duplicated stem issue will be dealt with separately. see issue #31
- There is still one unjustified dbh=0 that needs to be fixed, see issue #29
- For missed stem, I think we decided that you all will go through the ones that need to be changed to DN (when main stem is DN), the others are just waiting to be sampled (unfinished quadrat). Let me know if I am remembering that wrong.
- I'll address mortality census progress code (question of in progress vs completed, vs potentially new code and/or mortality map) in an new issue.
- There are still a few "CodeAButHOM1.3", I think some have notes that can help fixing the data. we may need to discuss this again.
@ValentineHerr
Copy link
Member Author

These were all addressed this morning but one (tag 52025, @krystalbagnaschi, looks like it was in your quadrat, 0517)

@ValentineHerr ValentineHerr changed the title unjustifiedZeroDBH for tag 10704 stemtag 2 unjustifiedZeroDBH May 9, 2023
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

No branches or pull requests

7 participants