Closed jpiaskowski closed 2 years ago
Here's my thoughts on this draft so far.
I think that
Ascotracer
should be "Diggle et al. 2002" not "Diggle 2002".Thank you! Can you take an editing round through it?
If the epidemiology section is not relevant, then no need to include it. It sounds like it would mislead R users into thinking there are useful packages in there for ag sci.
DSSAT was in there originally as a core package, but it's CRAN download stats are way lower than apsim, so I wondered how important it really is. Do you know any crop modellers to ask about this?
I was leaning towards a simplifed abbreviated reference style, "Author year", regardless of the number of authors. Would you be up for updating these references if this is important to you?
All good on the "Author year" style, I just hadn't caught that was the style. :)
I'll check, I do know that those are the two major crop/system models that are used.
I'll make a pass through the md file in the near future.
Perfect, thanks. I'll wait to implement the recommended changes from the CTV committee until you're done to avoid merge conflicts.
bumping this.
Sorry, I've not forgotten. Just busy with work.
I'm unsure about including GHCN. That data is akin to GSOD and NASA POWER, there is much overlap and these two are in the hydrology CTV (but GHCN isn't).
"cropgrowdays" is mentioned twice. Once on Line 16 and once on line 43, which is much more detailed and accurate, is this intended? The focus should be on evapotranspiration rather than the. weather data access I think.
I've submitted a pull request with very minor polishing leaving the more major issues for discussion here.
Thanks for catching the 'cropgrowdays' repeat. This was not intentional. We could make a pull request to Hydrology to add GHCN. Any more thoughts on DSSAT?
My apologies. I have been down with COVID off and on for the last month and it has put me wa-a-a-y behind on many things. I am in the lab all day today but will try to look at things tomorrow.
Janet
JANET WILLIAMS, Ph.D. Senior Research Scientist
College of Agricultural and Life Sciences Animal, Veterinary, and Food Sciences Dept Office: Ag Biotech 305 Lab: Ag Biotech 316 @.**@.> 208-885-6351 | 208-874-3213 (Cell) 875 Perimeter Dr MS2330 | Moscow ID 83844 | United States
From: Julia Piaskowski @.> Date: Thursday, August 4, 2022 at 8:16 AM To: IdahoAgStats/ctv-agriculture @.> Cc: Williams, Janet @.) @.>, Mention @.***> Subject: Re: [IdahoAgStats/ctv-agriculture] please review (Issue #31)
Thanks for catching the 'cropgrowdays' repeat. This was not intentional. We could make a pull request to Hydrology to add GHCN. Any more thoughts on DSSAT?
— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/IdahoAgStats/ctv-agriculture/issues/31*issuecomment-1205396824__;Iw!!JYXjzlvb!jjy-6D_g7A3KbC37L-iC9XN-kUy-2RYGeGKSx0uGvuUoXKaMFuXC8hk4EFY4YtLYnjk9BoghDq02cne6qmcZ2ZPOiA$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AAIRUB4YDB2O6LPMKPPPC7DVXPNELANCNFSM55BQZQAA__;!!JYXjzlvb!jjy-6D_g7A3KbC37L-iC9XN-kUy-2RYGeGKSx0uGvuUoXKaMFuXC8hk4EFY4YtLYnjk9BoghDq02cne6qmd3M57HLg$. You are receiving this because you were mentioned.Message ID: @.***>
I'm finishing up polishing to submit to the CTV committee.
@adamhsparks and @janetw
A draft of the CRAN task view is avaiable for feedback (The file called "Agriculture.md"). It's following the expected format for these task views.
Most (all?) other task views use a bulleted format, so I switched to that from the paragraphs (although, there a few paragraphs to consolidate information).
Can you please look at this, largely with copy editing in mind (e.g. misspellings, incorrect grammar, inconsistent style choices)? If you see a more substantial error, however, please bring it up.
Adam - can you please check the plant pathology section carefully? I made a few decisions about organization of that section that should be checked by an expert.
In the meantime, I will submit what is done thus far so the CTV committee can have a look. I created a new file for additional links so we can add them in the future. My plan is the delete the file called "draft_agriculture_ctv.md" because I don't want to maintain it. However, a close copy of that file is on the agstats.io website, which I plan to maintain for the foreseeable future.
Thank you!