Skip to content

Tools #27

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

Merged
merged 48 commits into from
Mar 9, 2020
Merged

Tools #27

merged 48 commits into from
Mar 9, 2020

Conversation

mrharpo
Copy link
Collaborator

@mrharpo mrharpo commented Feb 28, 2020

First round of TimelineTools and related functionality!!!

Added:

Plus numerous other fixes and improvements, including:

  • Better "mobile first" inline layout Mobile Swipe #15
  • Fixed duplicate clips on drag sort
  • Fixed time comparisons
  • Simplified component calculations
  • Added title text to Clips

And many more!

@normakster
Copy link

Cannot test due to the following:

The following error on the "dev" server caused by ... i/o error, open '/app/dist/Menu.map'.
Possible culprit: the /dist directory is "in use" or corrupted in the repo. Suggest removing and recreating. I've stop the services, restarted docker, restarted my machine, toggled branches and it's still stuck.

Logs:

$npm install
$docker-compose -f docker-compose.yml -f dev.yml up --build

[... build successful output omitted ...]

Starting alfred_dev_1 ... done                                   
Starting alfred_api_1 ... done                                   
Attaching to alfred_api_1, alfred_dev_1
api_1  | Checking for script in /app/prestart.sh
api_1  | Running script /app/prestart.sh
api_1  | Running inside /app/prestart.sh, you could add migrations to this file, e.g.:
api_1  |
api_1  | #! /usr/bin/env bash
api_1  |
api_1  | # Let the DB start
api_1  | sleep 10;
api_1  | # Run migrations
api_1  | alembic upgrade head
api_1  |
api_1  | INFO: Uvicorn running on http://0.0.0.0:80 (Press CTRL+C to quit)
api_1  | INFO: Started reloader process [1]
api_1  | INFO: Started server process [7]
api_1  | INFO: Waiting for application startup.
dev_1  | events.js:182
dev_1  |       throw er; // Unhandled 'error' event
dev_1  |       ^
dev_1  |
dev_1  | Error: EIO: i/o error, open '/app/dist/Menu.map'
alfred_dev_1 exited with code 1

@mrharpo
Copy link
Collaborator Author

mrharpo commented Mar 6, 2020

Fixes #5 - tools palette differentiates between actions

@mrharpo
Copy link
Collaborator Author

mrharpo commented Mar 6, 2020

Partial fix for #12 - export function now downloads rendered file.

@normakster
Copy link

Cannot test due to the following:

The following error on the "dev" server caused by ... i/o error, open '/app/dist/Menu.map'.
Possible culprit: the /dist directory is "in use" or corrupted in the repo. Suggest removing and recreating. I've stop the services, restarted docker, restarted my machine, toggled branches and it's still stuck.

Fixed the error by making the "api" service depend_on: - dev in the docker-compose.yml

@normakster
Copy link

normakster commented Mar 6, 2020

Ran into a couple issues related to missing media files. Do we have error handling slated for this?

api_1  | INFO: ('192.168.48.1', 33748) - "GET /videos/train.mp4 HTTP/1.1" 500
api_1  | ERROR: Exception in ASGI application
api_1  | Traceback (most recent call last):
api_1  |   File "/usr/local/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py", line 375, in run_asgi
api_1  |     result = await app(self.scope, self.receive, self.send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/fastapi/applications.py", line 140, in __call__
api_1  |     await super().__call__(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/applications.py", line 134, in __call__
api_1  |     await self.error_middleware(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 178, in __call__
api_1  |     raise exc from None
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 156, in __call__
api_1  |     await self.app(scope, receive, _send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 73, in __call__
api_1  |     raise exc from None
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 62, in __call__
api_1  |     await self.app(scope, receive, sender)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 590, in __call__
api_1  |     await route(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 208, in __call__
api_1  |     await self.app(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 44, in app
api_1  |     await response(scope, receive, send)
api_1  |   File "./partial.py", line 88, in __call__
api_1  |     raise RuntimeError(f"File at path {self.path} does not exist.")
api_1  | RuntimeError: File at path /app/videos/train.mp4 does not exist.
api_1  | INFO: ('192.168.48.1', 33758) - "GET /projects HTTP/1.1" 200
api_1  | INFO: ('192.168.48.1', 33758) - "GET /external.csv HTTP/1.1" 200
api_1  | INFO: ('192.168.48.1', 33758) - "GET /videos/train.mp4 HTTP/1.1" 500
api_1  | ERROR: Exception in ASGI application
api_1  | Traceback (most recent call last):
api_1  |   File "/usr/local/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py", line 375, in run_asgi
api_1  |     result = await app(self.scope, self.receive, self.send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/fastapi/applications.py", line 140, in __call__
api_1  |     await super().__call__(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/applications.py", line 134, in __call__
api_1  |     await self.error_middleware(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 178, in __call__
api_1  |     raise exc from None
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 156, in __call__
api_1  |     await self.app(scope, receive, _send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 73, in __call__
api_1  |     raise exc from None
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 62, in __call__
api_1  |     await self.app(scope, receive, sender)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 590, in __call__
api_1  |     await route(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 208, in __call__
api_1  |     await self.app(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 44, in app
api_1  |     await response(scope, receive, send)
api_1  |   File "./partial.py", line 88, in __call__
api_1  |     raise RuntimeError(f"File at path {self.path} does not exist.")
api_1  | RuntimeError: File at path /app/videos/train.mp4 does not exist.
api_1  | ffmpeg version 3.4.6 Copyright (c) 2000-2019 the FFmpeg developers
api_1  |   built with gcc 6.4.0 (Alpine 6.4.0)
api_1  |   configuration: --prefix=/usr --enable-avresample --enable-avfilter --enable-gnutls --enable-gpl --enable-libass --enable-libmp3lame --enable-libvorbis --enable-libvpx --enable-libxvid --enable-libx264 --enable-libx265 --enable-libtheora --enable-libv4l2 --enable-postproc --enable-pic --enable-pthreads --enable-shared --enable-libxcb --disable-stripping --disable-static --disable-librtmp --enable-vaapi --enable-vdpau --enable-libopus --disable-debug
api_1  |   libavutil      55. 78.100 / 55. 78.100
api_1  |   libavcodec     57.107.100 / 57.107.100
api_1  |   libavformat    57. 83.100 / 57. 83.100
api_1  |   libavdevice    57. 10.100 / 57. 10.100
api_1  |   libavfilter     6.107.100 /  6.107.100
api_1  |   libavresample   3.  7.  0 /  3.  7.  0
api_1  |   libswscale      4.  8.100 /  4.  8.100
api_1  |   libswresample   2.  9.100 /  2.  9.100
api_1  |   libpostproc    54.  7.100 / 54.  7.100
api_1  | [concat @ 0x7fa6be03d460] Impossible to open '/app/http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/BigBuckBunny.mp4'
api_1  | /app/dist/edl.edl: No such file or directory
api_1  | INFO: ('192.168.48.1', 33762) - "GET /render?edl=external.csv HTTP/1.1" 200
api_1  | INFO: ('192.168.48.1', 33762) - "GET /download?filename=external.csv.mp4 HTTP/1.1" 500
api_1  | ERROR: Exception in ASGI application
api_1  | Traceback (most recent call last):
api_1  |   File "/usr/local/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py", line 375, in run_asgi
api_1  |     result = await app(self.scope, self.receive, self.send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/fastapi/applications.py", line 140, in __call__
api_1  |     await super().__call__(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/applications.py", line 134, in __call__
api_1  |     await self.error_middleware(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 178, in __call__
api_1  |     raise exc from None
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/middleware/errors.py", line 156, in __call__
api_1  |     await self.app(scope, receive, _send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 73, in __call__
api_1  |     raise exc from None
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/exceptions.py", line 62, in __call__
api_1  |     await self.app(scope, receive, sender)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 590, in __call__
api_1  |     await route(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 208, in __call__
api_1  |     await self.app(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/routing.py", line 44, in app
api_1  |     await response(scope, receive, send)
api_1  |   File "/usr/local/lib/python3.7/site-packages/starlette/responses.py", line 253, in __call__
api_1  |     raise RuntimeError(f"File at path {self.path} does not exist.")
api_1  | RuntimeError: File at path videos/external.csv.mp4 does not exist.```

Copy link

@normakster normakster left a comment

Choose a reason for hiding this comment

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

No merge conflicts and it adds the functionality. We will need to open some issues around error handling of missing files.

Runs on Windows and Firefox.

@mrharpo
Copy link
Collaborator Author

mrharpo commented Mar 7, 2020

Sample files can be downloaded with ./alfred demo.

Maybe we tie this in during ./alfred install? The wget -N option skips existing media already, so we're safe to re run it without re downloading files.

@mrharpo mrharpo merged commit 92a87d2 into master Mar 9, 2020
@mrharpo mrharpo mentioned this pull request Mar 9, 2020
@mrharpo mrharpo deleted the tools branch June 27, 2020 00:45
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.

None yet

3 participants