q02/pool.v0.trr379.de-knowledge
10
0
Fork 0
Code Issues 3 Pull requests 2 Projects Releases Packages Wiki Activity Actions

Change DataItem part_of values to link to the dataset #4

Open
msz wants to merge 1 commit from pidtweak into main
pull from: pidtweak
merge into: q02:main
Conversation 0 Commits 1 Files changed 1 +39 -39
msz commented 2025-11-14 18:58:22 +00:00
Member
Copy link

After datalad dataset id convention was changed in 3590aca, the old IDs remained in DataItems. This does the following replacement:

old - "part_of":"datalad:698cec3c-e424-47e0-9da4-7a75e5ed6078"
new - "part_of":"datalad:698cec3c-e424-47e0-9da4-7a75e5ed6078/dl/dataset"

AFAICS the TRR379DataItem still has the part_of slot with TRR379Dataset range, so I am assuming this was just an ommission.

And although the relation could be deduced from the identifiers alone (overlapping part with the dataset), it would not be a relation in the linked-data sense, so I am assuming the part_of is still valid.

After datalad dataset id convention was changed in 3590aca, the old IDs remained in DataItems. This does the following replacement: ``` old - "part_of":"datalad:698cec3c-e424-47e0-9da4-7a75e5ed6078" new - "part_of":"datalad:698cec3c-e424-47e0-9da4-7a75e5ed6078/dl/dataset" ``` AFAICS the TRR379DataItem still has the `part_of` slot with TRR379Dataset range, so I am assuming this was just an ommission. And although the relation *could* be deduced from the identifiers alone (overlapping part with the dataset), it would not be a relation in the linked-data sense, so I am assuming the `part_of` is still valid.
After datalad dataset id convention was changed in 3590aca, the old IDs
remained in DataItems. This does the following replacement:

old - "part_of":"datalad:698cec3c-e424-47e0-9da4-7a75e5ed6078"
new - "part_of":"datalad:698cec3c-e424-47e0-9da4-7a75e5ed6078/dl/dataset"

AFAICS the TRR379DataItem still has the `part_of` slot with
TRR379Dataset range, so I am assuming this was just an ommission.
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin pidtweak:pidtweak
git switch pidtweak

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff pidtweak
git switch pidtweak
git rebase main
git switch main
git merge --ff-only pidtweak
git switch pidtweak
git rebase main
git switch main
git merge --no-ff pidtweak
git switch main
git merge --squash pidtweak
git switch main
git merge --ff-only pidtweak
git switch main
git merge pidtweak
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
q02/pool.v0.trr379.de-knowledge!4
No description provided.