clamsproject / aapb-annotations

Repository to store manual annotation dataset developed for CLAMS-AAPB collaboration
3 stars 0 forks source link

Template for future Readmes #46

Closed jarumihooi closed 7 months ago

jarumihooi commented 9 months ago

Because

we want the project readmes to all have the same structure (and they seem to be adaptable that), we will create a template for the readmes that each readme will follow.

Currently, the main issue is the discussion of the category names with @keighrim

We will use this branch's slate readme as the guide for the template.

==== Question: Can we have optional or differences in formatting?

# Official Project Name (Caps no dashes)

## Project Overview

### (Domain Knowledge Term Explanation optional?) <- right here, is this ok as optional?
About the project goals.

### Specs
* Annotation project name

Done when

Additional context

No response

keighrim commented 9 months ago

This might be a good starting point;

$ grep -P "^#+ " january-slates/readme.md 
# Slates
## Project Overview
### What is a film slate? 
### Specs
## Tool Installation: None
## Annotation Guidelines
### Preparation
### What to Annotate
### How to Annotate It 
### Decisions, Differentiations, and Precision during Annotation
## Data format and `process.py`
### `raw` data
### [`process.py`](process.py)
### `golds` data
## See also 
### 2020 Evaluation Dataset   
keighrim commented 9 months ago

and here's the template file from the previous PR. https://github.com/clamsproject/aapb-annotations/pull/47/commits/9904589039b142cfb0028ee32c03601faf1bc495 .

jarumihooi commented 9 months ago

Thanks I have a copy as well, is there something you want done with the grep'd version?

Still have this open question also: Can we have optional topic-lines? eg. what is a slate/phenomenon? -> optional?

keighrim commented 9 months ago

Unless we automatically "validate" the readme files based on some formal language (XML, jsonscheme, etc), I don't think we can actually force writers to use very specific sections or their title. For now I'm considering this template as more like a general guideline, rather than a hard restrictions.

jarumihooi commented 9 months ago

Ive updated the wording on the issue to specify that the current and inprogress readmes should be changed to conform to this (as a guideline).

I believe the recently merged issue #53 fixes the first bullet. Manually updating.

jarumihooi commented 7 months ago

The second bullet was handled piecemeal by commits and revisions to the readmes over a few PRs: https://github.com/clamsproject/aapb-annotations/pull/69/commits/03672c8b36b13a62b8f2f4ad7a9bd9f8f515b4fb from #69 , #60 , #64 , #59 , #55