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.

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

Torch #1

Open
wants to merge 81 commits into
base: main
Choose a base branch
from
Open

Torch #1

wants to merge 81 commits into from

Conversation

stroblme
Copy link
Member

  • added and manually optimized pytorch autoencoder (tensorflow AE can be launched by triggering dedicated pipeline)
  • splitted single pipeline into preprocessing, training and postprocessing
  • added poetry
  • added ssim metric replacing the pixel-wise accuracy
  • switched from matplotlib to plotly framework

stroblme added 30 commits June 5, 2023 16:53
@stroblme
poetry added
5f76c4a
@stroblme
torch and kedro viz
26e32da
@stroblme
pytorch fw
afec041
@stroblme
comments regarding data content
47bbc05
@stroblme
pytorch ae model
030e2d0
@stroblme
pt training
67b1a53
@stroblme
framework select param
7b8c166
@stroblme
default kedro run
cda2f84
@stroblme
disabled model saving
91cc8f3
@stroblme
data processing and metrics
ea9321b
@stroblme
data processing
8a73b61
@stroblme
using torchmetrics
145eb8a
@stroblme
added torchmetrics
7500042
@stroblme
some cleanup
fbc3496
@stroblme
added comments and exceptions
df5a52d
@stroblme
more comments
43e1ff2
@stroblme
added tensorflow ssim
da86adb
@stroblme
adapted for ssim
19fd8e4
@stroblme
refactoring for ssim
6add56e
@stroblme
serializable for exporting model
b731e90
@stroblme
common batch size def
9ccb6a8
@stroblme
refactoring regarding ssim
15c4267
@stroblme
refactoring
deb3b85
@stroblme
disabled console printing
d4e577c
@stroblme
tuned ae structure
216f0ec
@stroblme
reduced python req
4ac8ca4
@stroblme
experimental ae
c97bf45
@stroblme
quotes for python 3.9 compatibility
2d07085
@stroblme
added plotly
bae1c7e
@stroblme
switched to plotly fw
7c68617
eileen-kuehn
eileen-kuehn reviewed Jun 19, 2023
View reviewed changes
pyproject.toml Outdated Show resolved Hide resolved
eileen-kuehn
eileen-kuehn requested changes Jun 23, 2023
View reviewed changes
Copy link
Member

@eileen-kuehn eileen-kuehn left a comment

Choose a reason for hiding this comment

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

There are some smaller comments and suggestions. When those are done, we are good to go 👍

pyproject.toml Outdated Show resolved Hide resolved
src/qc_data_preparation/helpers/dataset.py Show resolved Hide resolved
src/qc_data_preparation/helpers/dataset.py Outdated Show resolved Hide resolved
src/qc_data_preparation/helpers/types.py Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
@eileen-kuehn eileen-kuehn mentioned this pull request Jun 24, 2023
Closed
@eileen-kuehn eileen-kuehn mentioned this pull request Jun 25, 2023
Closed
eileen-kuehn and others added 4 commits June 25, 2023 23:34
@eileen-kuehn @maxfischer2781
feat: add abstractdataset to save and load pytorch and tensorflow mod…
8fd5625 
…els (#7)

* feat: add model class to handle torch and tensorflow

* build: specify version for tensorflow

* feat: introduce number_of_features globally

---------

Co-authored-by: Max Fischer <[email protected]>
@eileen-kuehn
fix: fix functioning to store plotly html (#8)
c8e2b4c
@eileen-kuehn
fix: fix intendation of error raised
67bc13d
@stroblme
removed dependency on pt and tf model import
f48b8d1
@stroblme stroblme requested a review from eileen-kuehn July 3, 2023 14:38
@stroblme stroblme mentioned this pull request Aug 10, 2023
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eileen-kuehn eileen-kuehn Awaiting requested review from eileen-kuehn

Requested changes must be addressed to merge this pull request.

Assignees

@stroblme stroblme

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stroblme @eileen-kuehn @ulivc