Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KBI0013: Merge Data Availability Information #45

Merged
merged 7 commits into from May 5, 2023
Merged

KBI0013: Merge Data Availability Information #45

merged 7 commits into from May 5, 2023

Conversation

loj
Copy link
Contributor

@loj loj commented Apr 26, 2023

This is a start of a KBI for #28 using @mih's notes from datalad/datalad#600 (comment)

Right now it only covers the use-case of splitting into multiple subdatasets. I plan to come back to this on my next support shift to explore adding a few other examples of how this workflow can be used.

kbi/0013/index.rst Outdated Show resolved Hide resolved
Comment on lines 6 to 7
KBI0013: Merge Data Availability Information
============================================
Copy link
Contributor

Choose a reason for hiding this comment

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

I feel like the title should somehow reflect the context of the text below (splitting a dataset). Right now it is really broad. I am not sure would would be best. Two possible directions

  • Split a dataset and reuse data hosting
  • Import content availability information on shared annex keys from another dataset

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree that "Merge Data Availability Information" was broad.

I'm still not crazy about the current title, but I wasn't able to come up with anything better.

loj and others added 5 commits April 28, 2023 12:21
Co-authored-by: Adina Wagner <adina.wagner@t-online.de>
Suggested by: Michael Hanke <michael.hanke@gmail.com>
Show the workflow with 2 files and 2 targets.
Link to the handbook workflow for copy-file instead of to the
datalad-docs.

Suggested-by: Adina Wagner <adina.wagner@t-online.de>
@loj
Copy link
Contributor Author

loj commented Apr 28, 2023

I've made a few updates and chose to keep the focus on splitting datasets.

@adswa adswa linked an issue Apr 28, 2023 that may be closed by this pull request
4 tasks
Copy link
Contributor

@adswa adswa left a comment

Choose a reason for hiding this comment

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

This works for me, thanks for writing it up! I left a few minor suggestions, but feel free to ignore them - when I read this KBI being unfamiliar with the command, I wondered what format this export was in and read it up; the suggestions add rough and short comments about this.

kbi/0013/index.rst Show resolved Hide resolved
kbi/0013/index.rst Outdated Show resolved Hide resolved
kbi/0013/index.rst Outdated Show resolved Hide resolved
Co-authored-by: Adina Wagner <adina.wagner@t-online.de>
Copy link
Contributor

@mih mih left a comment

Choose a reason for hiding this comment

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

Works for me, thx!

@mih mih merged commit b8572f9 into main May 5, 2023
3 checks passed
@mih mih deleted the 0013 branch May 5, 2023 05:54
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.

KBI on merging availability information with existing tooling
3 participants