Skip to content
This repository has been archived by the owner on Aug 3, 2020. It is now read-only.

Possible error with model implementation #3

Open
maxzuo opened this issue Nov 3, 2018 · 1 comment
Open

Possible error with model implementation #3

maxzuo opened this issue Nov 3, 2018 · 1 comment
Assignees
Labels
help wanted Extra attention is needed invalid This doesn't seem right

Comments

@maxzuo
Copy link
Collaborator

maxzuo commented Nov 3, 2018

Model taking too long to run on iPhone. Testing other models out there of larger sizes, the model should not take as long as it does to make a prediction.

Possible sources of problem:

  • Model created in Keras not done properly (issue with architecture).
  • Model implementation within Swift not optimized
  • Model size too large.
@maxzuo maxzuo added bug Something isn't working help wanted Extra attention is needed labels Nov 3, 2018
@maustinstar maustinstar added invalid This doesn't seem right and removed bug Something isn't working labels Nov 7, 2018
@maustinstar maustinstar removed this from the Computational Media Showcase milestone Nov 17, 2018
@maxzuo
Copy link
Collaborator Author

maxzuo commented Nov 30, 2018

Issue with architecture resolved by switching to almost an entirely new architecture.

  • SRCNN went too deep too quickly. By adding more convolutional layers but gradually reaching 64 filters, the new model can help decrease the amount of time spent.
  • New model's PSNR accuracy should also be improved.
  • Further experimentation with PyTorch required. Model speed still not optimal.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted Extra attention is needed invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

2 participants