docs(deposits): state that deposits/ mirrors PSDI's latest version

Open Beginner friendly
#77 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
92/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Active
Domain
documentation

Research direction

Open deposits/README.md and review how directories and published model versions are currently described. Document the one-directory-per-record, update-in-place convention, including that superseded versions remain in git history, then verify the acceptance criteria before #70 publishes another version.

Written by the indexing model from the issue text.

Description

Problem

deposits/ holds one directory per published model, and the convention has
always been that a directory describes the record's current PSDI version —
the CGCNN v2 release replaced deposits/metallicity/is_metal/cgcnn in place
rather than adding a sibling directory. That convention was never written
down anywhere.

The 1-based k-index migration (#70) is about to publish a second version of
d5ds2-64f16. Without the rule stated, the obvious move is a v1/v2
directory split, which would leave the repository holding two descriptions
of one record, only one of which is true.

Proposed approach

Document the convention in deposits/README.md: one directory per record,
describing its current version; on a new PSDI version, update the directory
in place (new checksums, new numbers, same path); superseded versions stay in
git history, not as a sibling directory.

Acceptance criteria

  • deposits/README.md states the one-directory-per-record, update-in-place
    rule before #70 produces a second version of any record.

Written by an agent on behalf of Junwen Yin.

Dominant language
Python
Stars
1
Forks
1
Avg merge
3h 17m
Merged PRs (30d)
55

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from stfc/goldilocks-ml

All issues in stfc/goldilocks-ml

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.