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

Demo notebook #139

Merged
merged 1 commit into from
Feb 3, 2025
Merged

Demo notebook #139

merged 1 commit into from
Feb 3, 2025

Conversation

FelipeTrost
Copy link

Demo notebook for shell

Addresses #135

I had to rewrite many absolute imports to be relative, because they didn't work in the built version of the sdk.

The deliverables were deleted from develop, when I started this branch they weren't, that is why they are being added back in.

@FelipeTrost FelipeTrost requested a review from Timm638 February 3, 2025 13:04
@Timm638
Copy link

Timm638 commented Feb 3, 2025

Looked good except the circular import error (by trying to import MachineLearningInterface from the Forecasting Module ,which was a parent from the importing module, instead of the interfaces File).
Merged into Demo Notebook Branch.

@Timm638 Timm638 closed this Feb 3, 2025
@Timm638 Timm638 reopened this Feb 3, 2025
@Timm638 Timm638 changed the base branch from develop to feature/096_rtdip_demo_pipeline February 3, 2025 16:56
@Timm638 Timm638 merged commit c3e91f9 into feature/096_rtdip_demo_pipeline Feb 3, 2025
14 checks passed
@Timm638 Timm638 deleted the demo-notebook branch February 3, 2025 17:16
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