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.

Sign up for GitHub

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

Jump to bottom

Organize script structure, rewrite properly #12

Open

1 of 4 tasks

AlexisRenchon opened this issue Dec 9, 2021 · 1 comment · Fixed by #36, #39 or #40

Open

1 of 4 tasks

Organize script structure, rewrite properly #12

AlexisRenchon opened this issue Dec 9, 2021 · 1 comment · Fixed by #36, #39 or #40

Assignees

Labels

Best practice Organisation

Milestone

CUP of TEA v0.2

Member

AlexisRenchon commented Dec 9, 2021 •

edited

Loading

/functions/ should contain only folders, no scripts - started in Tweaks #36 , done in restructure #39
Commented MWE below every functions, started in functions MWE #40
Rewrite menu1 (interactive DAMM) clean, using DAMMmodel.jl, with the julia script section in /functions/MAIN/
Create TEA.jl package to load and filter databases such as FLUXNET or COSORE

Before going further, take a step back after FLUXNET respiration figure.
reorganize folders structure in /functions, make sure each script is tidy.

AlexisRenchon added the Organisation label

AlexisRenchon added this to the CUP of TEA v0.2 milestone

AlexisRenchon pinned this issue

AlexisRenchon added the Best practice label

Member Author

AlexisRenchon commented Dec 14, 2021

TO DO:
don't use include() inside functions
function script only contain functions. no using and no include

AlexisRenchon self-assigned this

This was linked to pull requests Dec 22, 2021

Merged

restructure #39

Merged

functions MWE #40

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment