Skip to content

Conversation

@IsaiMaganTNO
Copy link
Contributor

Related issues

This pull request is related to several issues, namely reconstructing a Workflow tutorial for the TLC, showing a case where multiple runs of TEM are done through one connection and working with explicit DuckDB files to store results, read in data etc.

Old PR is closed and this new one contains (for now) finalized version of TLC Workshop 7 and squashed to keep commit history clean.

Checklist

  • I am following the contributing guidelines
  • Tests are passing
  • Lint workflow is passing
  • Docs were updated and workflow is passing

@IsaiMaganTNO IsaiMaganTNO requested review from clizbe and gnawin October 27, 2025 13:17
Copy link
Member

@clizbe clizbe left a comment

Choose a reason for hiding this comment

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

I don't have time to follow the whole thing, but I guess the TLC will be a good road-test.
But HOLY CRAP THIS LOOKS AWESOME! 8D
Great job! I really like that it comes from multiple file formats and is using actual data people might use. Fantastic!

@datejada datejada force-pushed the tlc-workflow-tutorial branch from 57809f3 to d4c5b92 Compare November 26, 2025 15:14
@datejada
Copy link
Member

@IsaiMaganTNO I have updated the branch to the latest changes in main to prepare to merge it. Before that, would you mind double-checking that all works as you expected in the tutorial? You need to update your local branch to get the changes I added. Specially, the names of the parameters that have changed, please have a look at #1432, Thanks, and let me know if you need some guidance or assistance with this.

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.

3 participants