Skip to content

fixed link to agronomic metadata tutorial#35

Merged
dlebauer merged 13 commits intoterraref:masterfrom
kimberlyh66:master
Nov 7, 2018
Merged

fixed link to agronomic metadata tutorial#35
dlebauer merged 13 commits intoterraref:masterfrom
kimberlyh66:master

Conversation

@kimberlyh66
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

@dlebauer dlebauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - thanks for catching that! Please see comment - I think there is an errant 6 at the top of the file

Comment thread traits/01-web-access.Rmd Outdated
@dlebauer dlebauer self-assigned this Nov 7, 2018
Co-Authored-By: kimberlyh66 <44081116+kimberlyh66@users.noreply.github.com>
Comment thread traits/02-betydb-api-access.Rmd Outdated

```sh
wget -O sorghum.json
"https://terraref.ncsa.illinois.edu/bety/api/beta/species?genus=Sorghum&key=9999999999999999999999999999999999999999"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

put the \ in (\\ doesn't work), as well as the comment.

please replace all wget -O with curl -o to make it more platform independent (e.g. work on macs where wget isn't available by default)

Comment thread traits/02-betydb-api-access.Rmd Outdated
"https://terraref.ncsa.illinois.edu/bety/api/beta/species?genus=Sorghum&key=`cat ~/.betykey_public`"
```

> Do you know what `cat ~/.betykey_public`. Run the command and see what it outputs in your shell.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can delete this stuff

Comment thread traits/04-danforth-indoor-phenotyping-facility.Rmd Outdated
Copy link
Copy Markdown
Member

@dlebauer dlebauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@dlebauer dlebauer merged commit 3c5a154 into terraref:master Nov 7, 2018
kimberlyh66 and others added 2 commits November 7, 2018 15:24
Co-Authored-By: kimberlyh66 <44081116+kimberlyh66@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

2 participants