Skip to content

new properties (license, version) from citation.cff. Fixes #935#948

Merged
dgarijo merged 20 commits intoKnowledgeCaptureAndDiscovery:devfrom
juanjemdIos:master
Apr 16, 2026
Merged

new properties (license, version) from citation.cff. Fixes #935#948
dgarijo merged 20 commits intoKnowledgeCaptureAndDiscovery:devfrom
juanjemdIos:master

Conversation

@juanjemdIos
Copy link
Copy Markdown
Contributor

No description provided.

@dgarijo
Copy link
Copy Markdown
Collaborator

dgarijo commented Apr 7, 2026

@juanjemdIos what about the info from the reference publication?

Comment thread docs/index.md Outdated
Comment thread docs/index.md Outdated
- **Build file**: Build file(s) of the project. For example, files used to create a Docker image for the target software, package files, etc.
- **Citation**: Preferred citation as the authors have stated in their readme file. SOMEF recognizes Bibtex, Citation File Format files and other means by which authors cite their papers (e.g., by in-text citation). We aim to recognize the following properties:
- **Citation**: Preferred citation(s) as the authors have stated in their readme file. SOMEF recognizes Bibtex, Citation File Format files and other means by which authors cite their papers (e.g., by in-text citation).
For CITATION.cff files, SOMEF now generates two separate entries: one for the software (is_preferred_citation: False) and another for the preferred citation (is_preferred_citation: True). This ensures metadata like DOI or version is correctly assigned to each entity.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

but if there is no preferred citation, then the software one is not false.
Instead of generating False, I would only leave is_preferred_citaiton: True when we find prefered citation. And do not add anything otherwise.

Comment thread docs/index.md
Comment thread docs/index.md Outdated
Comment thread docs/index.md Outdated
Comment thread docs/output.md Outdated
- `forks_url`: Links to forks made of the project (GitHub only)
- `full_name`: Name + owner (owner/name) (if available)
- `full_title`: If the repository has a short name, we will attempt to extract the longer version of the repository name. For example, a repository may be called "Widoco", but the longer title is "Wizard for documenting ontologies".
- `funding`: Funding code for the related project.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

wait, we don't do this

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please double check

Comment thread docs/output.md Outdated
|---|---|---|
| **name** | String | Name of the requeriment |
| **version** | String | named version of a requeriment |
| **dependency_type** | String | type: dev, runtime... Indicates whether the dependency is required at runtime or only for development/testing |
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

List all dependency types we support.
We support 3: development, runtime and documentation

Comment thread docs/output.md Outdated


A SOFTWARE_APPLICATION has the following properties:
A Software_application has the following properties:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This one does not make a lot of sense. What is the difference between this and Requirement?

Comment thread src/somef/test/test_codemeta_export.py Outdated
json_content = json.loads(data)

copyright_holder = json_content[constants.CAT_CODEMETA_COPYRIGHTHOLDER]
print(copyright_holder)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

do not print. Log or remove pls

Comment thread README.md Outdated
Copy link
Copy Markdown
Collaborator

@dgarijo dgarijo left a comment

Choose a reason for hiding this comment

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

Please see the required changes

Comment thread docs/index.md Outdated
Comment thread docs/index.md Outdated
Comment thread docs/index.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
@dgarijo dgarijo merged commit 121e647 into KnowledgeCaptureAndDiscovery:dev Apr 16, 2026
2 checks passed
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