-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
1 addition
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1df891d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rBatt Ryan, did I add this into the metadata correctly? I wanted to document the swept area for the NEUS biomass data.
1df891d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, the .xml files are generated from simpler R scripts; for example: https://github.com/mpinsky/OceanAdapt/blob/master/R/metadata/metaData.neus.R
The .xml files follow the Ecological Metadata Language (EML) format, and the scripts use the R package "EML".
I tried to use standardized definitions where applicable. The reason for this is to indicate to readers which columns might be comparable between data sets; if both column names and definitions are idiosyncratic, it can be difficult to follow. For this reason, I use the gen.cols&units.R script to provide template/ standardized units and definitions.
The item you've corrected is on the line with the text
"BIOMASS" = gen.cols[["weight"]]
; instead of subsettinggen.cols
, a custom definition could be written. There might be a way to add a note or comment to a data field (in addition to a description and units), that way we could keep the standard descriptions and units (so it is easier to compare data sets). But it's been a while since I did this, so I don't know/ forget.1df891d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1df891d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1df891d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1df891d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1df891d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1df891d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.