Skip to content

pop transformation pipeline#66

Merged
EstebanMontandon merged 3 commits intomainfrom
snt25-426
Apr 13, 2026
Merged

pop transformation pipeline#66
EstebanMontandon merged 3 commits intomainfrom
snt25-426

Conversation

@claude-marie
Copy link
Copy Markdown
Contributor

No description provided.

@EstebanMontandon
Copy link
Copy Markdown
Collaborator

Hey @claude-marie, could you take a look at the changes in my last commit? I think this is an example of how we should aim to organize code across our pipelines moving forward. Let me know what you think!

Please notice that I have separated the functions in the R file in two groups:
group 1 what should potentially be moved to SNT utils, and
group 2 (after the tag) for functions specific for the pipeline.

The notebook remains step by step in terms of the computation, for this pipeline I don't expect many people will try to modify the code , but if it happens , it will still be more or less understandable of what is going on.

@EstebanMontandon
Copy link
Copy Markdown
Collaborator

let's try to apply a standard

@EstebanMontandon EstebanMontandon merged commit b42c487 into main Apr 13, 2026
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.

2 participants