Skip to content

feat: ✨ add initial collaborate on GenomeDK section#385

Open
signekb wants to merge 7 commits into
mainfrom
feat/add-initial-collaborate-on-genomedk-section
Open

feat: ✨ add initial collaborate on GenomeDK section#385
signekb wants to merge 7 commits into
mainfrom
feat/add-initial-collaborate-on-genomedk-section

Conversation

@signekb

@signekb signekb commented Jul 7, 2026

Copy link
Copy Markdown
Member

Description

This adds some points about how to work together in a shared folder on GenomeDK that @martonvago mentioned when we talked earlier today. This might/probably will be elaborating on as we go, but wanted to add these points as a starting point.

Needs a quick review.

@signekb signekb moved this from Todo to In Review in Product development Jul 7, 2026

@lwjohnst86 lwjohnst86 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice start!

Comment thread how-we-work/genome-dk.qmd Outdated
Comment thread how-we-work/genome-dk.qmd

## Collaborating and working in a shared folder

When you work in a shared folder on GenomeDK, any changes you make (e.g.,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you include a concrete folder structure here for "shared folder"? Are you working in the exactly the same folder? I made a comment or issue about this I thought, but I can't find it. Do you work like:

onlimit-study/
└── feasibility-data/
    ├── main/
    ├── signe/
    └── marton/

So you have a clone in signe/ and Marton works in marton/? Or do you all work in feasibility-data/? If the second, we should probably work like the above to avoid this "shared editing" situation.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah, yes, it's the second situation; there's one feasibility-data/ that everyone works in. With the first, you mean we each have a branch, or that we have n_users + 1 clones of the repo (including the data)?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We each have clones (with our name for the clone), where we all push and pull from GitHub.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Do we then only have the checked-out data files in the main/ repo? Otherwise, we would have multiple copies of the raw data - one copy for each local repo - right?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should be solved by using the build and release process I wrote about. No data is saved outside of the main repo on genomedk

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Alright. I'll update this with my understanding of how we'll work 👍

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I had a look at this and wrote a comment.

Just two points here:

  • We don't even need to nest the individual clones under a feasibility-data/ folder, everyone can clone where they want. Or is there some benefit to having the clones in a shared folder?
  • I think Signe is right about the data duplication. If a clone wants to do something with the data, it has to smudge it, and then there will be duplication. If the clones never smudge, that is the same as working outside of GenomeDK and then I'm not sure what the clones on GenomeDK are for.

Comment thread how-we-work/genome-dk.qmd Outdated
@github-project-automation github-project-automation Bot moved this from In Review to In Progress in Product development Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

3 participants