Replies: 8 comments 1 reply
-
A few comments:
|
Beta Was this translation helpful? Give feedback.
-
|
Beta Was this translation helpful? Give feedback.
-
Notes from PEcAn meeting June 30 2025
We may need:
I have updated the spec1 to include
Next steps:
Footnotes |
Beta Was this translation helpful? Give feedback.
-
Below are some brief comments about the event format and the discussions that have already taken place.
|
Beta Was this translation helpful? Give feedback.
-
Hi David, Thank you for the spec, the general idea behind it and the proposed implementation look great to me. A few comments:
|
Beta Was this translation helpful? Give feedback.
-
@istfer and @mdietze - ICASA has just migrated from XLSX to CSV, does this allay concerns of machine-readability? |
Beta Was this translation helpful? Give feedback.
-
Conversion from XLSX to CSV does not by itself make the information machine-readable. Additional key criteria would be whether they’ve made the data “tidy” or not, and whether things are joined into one large table or still exist and many smaller tables. The latter is technically “machine readable” but may still be a major pain in the butt to work with (i.e. it doesn’t alleviate the extent to which expert knowledge is needed to be able to understand what info is in what tables and how to join them). This information doesn’t appear to be of sufficient size and complexity to really justify having to do complex database joins and lots of complex conditional logic in the coding (i.e. it strikes me as something that really could/should exist as a single tidy table)
On Jul 16, 2025, at 7:44 PM, David LeBauer ***@***.***> wrote:
@istfer<https://github.com/istfer> and @mdietze<https://github.com/mdietze> - ICASA has just migrated from XLSX to CSV<DSSAT/ICASA-Dictionary#5>, does this allay concerns of machine-readability?
—
Reply to this email directly, view it on GitHub<#3551 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAXQJ6TFFTLFFSWGCSEWOYL3IZ6NPAVCNFSM6AAAAACABIC5OGVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTGNZXHE4DOOI>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
I've opened a PR #3621 with an MVP events.json schema and a script that generates examples; and some examples. What it includes from this discussion:
This is what is still TODO (post MVP)
These are nice to have
Not included (per discussion)
|
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Description
PEcAn is adopting a 'standard' format for representing events including agronomic management and disturbance. This file type is intended to be analogous to the met data standard - an intermediate, model-independent representation that supports downstream translation into model-specific configurations.
Background
Though we have developed a format e.g.
fieldactivity_events.json
and JSON schema
ui_structure.json as part of the fieldactivity app app.
Details
Our use case is focused on agronomic events, but this could be extended to land use change, disturbance, etc.
Criteria:
Request for feedback
The proposed specification is at
https://gist.github.com/dlebauer/5a2e505bca6afe66ca6aa44d9dd163f7
Please react to this thread w/ the appropriate emoji
And comment as appropriate
We will discuss this at the next PEcAn biweekly, and I'd like to finalize this and begin implementing at that point.
cc @istfer @HenriKajasilta @mdietze
Beta Was this translation helpful? Give feedback.
All reactions