forc-db / IPCC-EFDB-integration

Creative Commons Attribution 4.0 International
4 stars 3 forks source link

add list of studies sent in each batch #28

Closed teixeirak closed 2 years ago

teixeirak commented 2 years ago

@ValentineHerr , Here's what we want:

"A summary list, in excel format, of all data sources; and for each data source the number of data submitted, clustered by carbon pool, with univocal reference to the relevant EFDB entry form in which data have been complied."

You started this in this commit.

ValentineHerr commented 2 years ago

@teixeirak and @mawilliams99 , does this table sound right?

teixeirak commented 2 years ago

Mostly, but they said they want it clustered by carbon pool... Would that be tricky to code? I wouldn't bother if it's tricky.

ValentineHerr commented 2 years ago

So, what does clustered by carbon pool would mean here ? Can you give an example?

Get Outlook for iOShttps://aka.ms/o0ukef


From: Kristina Anderson-Teixeira @.> Sent: Tuesday, September 28, 2021 6:21:54 PM To: forc-db/IPCC-EFDB-integration @.> Cc: Herrmann, Valentine @.>; Assign @.> Subject: Re: [forc-db/IPCC-EFDB-integration] add list of studies sent in each batch (#28)

External Email - Exercise Caution

Mostly, but they said they want it clustered by carbon pool... Would that be tricky to code? I wouldn't bother if it's tricky.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fforc-db%2FIPCC-EFDB-integration%2Fissues%2F28%23issuecomment-929668109&data=04%7C01%7Cherrmannv%40si.edu%7Cb7552443afdd4b16d18808d982ce610b%7C989b5e2a14e44efe93b78cdd5fc5d11c%7C0%7C0%7C637684645185285257%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=fenKUeFyjVYRLzX%2ByUc5kwruppeKyP8lzSYCCIySUlY%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEWDCINLK3KS4LQ7YSSKY63UEI5YFANCNFSM5EFPK53A&data=04%7C01%7Cherrmannv%40si.edu%7Cb7552443afdd4b16d18808d982ce610b%7C989b5e2a14e44efe93b78cdd5fc5d11c%7C0%7C0%7C637684645185295215%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=HcpNCe9SVcLDEHK1or6L%2BECvgtyW6aFVCjlOYOUrZUk%3D&reserved=0. Triage notifications on the go with GitHub Mobile for iOShttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04%7C01%7Cherrmannv%40si.edu%7Cb7552443afdd4b16d18808d982ce610b%7C989b5e2a14e44efe93b78cdd5fc5d11c%7C0%7C0%7C637684645185295215%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=4v%2F3j3%2FNfrSKIh9UqFIES88lC9Yu%2BW2svY9H7lTXeEE%3D&reserved=0 or Androidhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04%7C01%7Cherrmannv%40si.edu%7Cb7552443afdd4b16d18808d982ce610b%7C989b5e2a14e44efe93b78cdd5fc5d11c%7C0%7C0%7C637684645185295215%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=TS7YIQEUti7gyCsWablMKwPyGyWemJwJf71DD%2BT4FyU%3D&reserved=0.

teixeirak commented 2 years ago

Good question-- at first I envisioned a column for each variable indicating the number of records for each, but that's a lot of columns, so maybe just include a list of the variables (optionally with n records for each in parentheses) included in each batch.

ValentineHerr commented 2 years ago

"A summary list, in excel format, of all data sources; and for each data source the number of data submitted, clustered by carbon pool, with univocal reference to the relevant EFDB entry form in which data have been complied."

I could combine the number of data submitted by carbon pool in one same cell for each data source (or more likely each EFDB entry form), But I feel that it would get less readable/usable.

I'll leave it as is for now and and will return to it later. @mawilliams99, if we haven't found the best format by the time you send it to Valentyna, ask for feedback and I'll adjust the table accordingly.

teixeirak commented 2 years ago

Sounds good. I think we can go with this and modify if they request some other change.