ReproNim / OHBMEducation-2022

Repo for the OHBM Education 1/2 day course entitled: "How to Write a Re-Executable Publication"
http://www.repronim.org/OHBMEducation-2022/
Creative Commons Zero v1.0 Universal
3 stars 15 forks source link

datalad siblings github --publish-depends github-lfs eror... #10

Open dnkennedy opened 2 years ago

dnkennedy commented 2 years ago

In the publish section we are told to run:

datalad siblings github --publish-depends github-lfs

I get the following error:

usage: datalad siblings [-h] [-d DATASET] [-s NAME] [--url [URL]] [--pushurl PUSHURL] [-D DESCRIPTION]
                        [--fetch] [--as-common-datasrc NAME] [--publish-depends SIBLINGNAME]
                        [--publish-by-default REFSPEC] [--annex-wanted EXPR] [--annex-required EXPR]
                        [--annex-group EXPR] [--annex-groupwanted EXPR] [--inherit] [--no-annex-info] [-r]
                        [-R LEVELS] [--version]
                        [{query|add|remove|configure|enable}]
datalad siblings: error: argument {query|add|remove|configure|enable}: invalid constraint:{'query', 'add', 'remove', 'configure', 'enable'} value: 'github'

I think this step was updated by @jsheunis ; any ideas what I'm missing?

yarikoptic commented 2 years ago

what publish section -- reference? may be you missed configure and -s? look at usage error gave you -- it is there to help!

datalad siblings configure -s github --publish-depends github-lfs
dnkennedy commented 2 years ago

reopening to get the correction into the instructions...

dnkennedy commented 2 years ago

This is still a 'problem'. Currently, the instructions work, but the GitHub repo get's left with the git-annex branch being the default, so that the user has to manually reset that to 'master' being the default, or else datalad install doesn't work correctly.

@yarikoptic questioned the datalad version (mine is 0.16.2) (or blamed me for using a mac...), but I'm still awaiting a resolution as to how to get this to work completely correctly...

jsheunis commented 2 years ago

@dnkennedy this sounds like a git/github default branch issue.

Any new DataLad dataset configured with master instead of main as the default branch will get git-annex configured to be the default displayed branch when it is pushed to GitHub. See this FAQ for more information. This can be prevented by:

yarikoptic commented 2 years ago

FWIW, submitted a potential fix to this mess (never enough tests, isn't it!!!?) https://github.com/datalad/datalad/pull/6750