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

Update folder structure #37

Closed
sofiasiamp opened this issue Jan 29, 2025 · 0 comments · Fixed by #45
Closed

Update folder structure #37

sofiasiamp opened this issue Jan 29, 2025 · 0 comments · Fixed by #45
Assignees
Labels
enhancement New feature or request

Comments

@sofiasiamp
Copy link
Member

sofiasiamp commented Jan 29, 2025

One more thing that I have noticed lately and that we may want to change sooner rather than later: folder structure. With the increase in folders, the project has become quite messy and it is not easy any more to find the relevant ones quickly.

Besides the four folders that are necessary for the studies, we could have 1 folder "Analyst" or "Analyst_Space" (sth like that) that will include the Tracy, Ines and Franzi folders.

And another dedicated folder that incorporates all the utility that we build and typically only we are using (perhaps "utils" or "utility"): create_mock_data, tests, workflow.

I know we need to re-check pathing again for the changes just made, but I think it will be worth it.

In that sense, I would then create a "compare" (working title) folder under utility which stores the csv files (with according sub-folder structure) for comparing git diffs and the function to create them. And under "Analysts" we can have the simple R Script calling that function.

What do you think?

Originally posted by @FlorianSchw in #36 (comment)

@sofiasiamp sofiasiamp added the enhancement New feature or request label Jan 29, 2025
@sofiasiamp sofiasiamp self-assigned this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant