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

error in Makiranta_2012_tiol #239

Closed
teixeirak opened this issue May 26, 2021 · 2 comments
Closed

error in Makiranta_2012_tiol #239

teixeirak opened this issue May 26, 2021 · 2 comments
Labels

Comments

@teixeirak
Copy link
Member

Krista & Ben, 

 

With Krista we have emailed a lot a few years ago and with Ben related to the Palviainen et al. paper from last year as we were both coauthors.

 

Thanks for the great work in expanding ForC-database and congratulations for the two recent papers, in Environmental Research Letters and Global Change Biology which we have read very carefully.

 

We, together with postdoc Xia Chen, are starting to model succession from physiological principles based on ForC. I noticed that a datapoint in Finland that I checked had a very high biomass value:

x | y | Forest.type | Stand.age.years. | AGB.t.ha. | Reference -- | -- | -- | -- | -- | -- 25.11667 | 61.36667 | Natural | 55 | 281.702128 | Makiranta_2012_tiol

I checked from the original publication and even asked one of the authors and we do not understand from where the value of 281 Mg/ha comes from. 

 

Is there another way to report suspicious data values in ForC or is this the way? This is just one row of data but I am afraid that e.g. some rows have jumped and several rows are now having wrong values so it could be wise to trace the source of this potential error. Xia checked and Ben's name was mentioned as the one entering the data.

 

Regards,
Markku

 

-- 

 

Markku LARJAVAARA

Peking University

@teixeirak teixeirak added the bug label May 26, 2021
teixeirak added a commit that referenced this issue May 26, 2021
@teixeirak
Copy link
Member Author

I flagged the value as suspicious so that it won’t get used in future analyses, and will review when I have a chance.

@bpbond , flagging you here FYI.

@teixeirak
Copy link
Member Author

This value itself was correct, but value is total biomass (aboveground and belowground), and units are dry organic matter, not C.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant