Commit Graph

140 Commits (e14afdb6c1af2dd6b756fe6465cc5fa692cb15d0)

Author SHA1 Message Date
Christopher Usher add8d021a8 also apply fix to get_thumbnail 6 days ago
Mike Lang 861a426395 thrimshim: Fix performance issue with getting template png
Bytes returned from the database are a "memoryview" and not a bytes object.
These mostly behave the same, but one difference is that Flask recognizes
a bytes object as something it can send as-is, whereas a memoryview ends up using
its generic "iterable" processing. This results in sending every individual byte of the result
as a single part of a HTTP chunked encoding response, adding 5 bytes and a syscall to every byte returned.

The solution is to explicitly convert it to bytes before returning.
6 days ago
ElementalAlchemist 501549350b Fix name duplicate check to only run when the template name is changed
Without checking first, a rename is required for all thumbnail template modifications.
1 week ago
ElementalAlchemist ea6d256b6d Fix database access 1 week ago
Christopher Usher 4f5805fd8d I thought we had fixed this 1 week ago
ElementalAlchemist 8ac23f2927 When checking whether a template exists, check the right table 1 week ago
Christopher Usher b4882a9fc4 More bug fixes 1 week ago
Christopher Usher 7ed0bbedc4 Fixed typo 1 week ago
Christopher Usher d6f976ac0f Bug fixes and improvements in response to comments on the PR 1 week ago
Christopher Usher ecd638356d thrimshim now uses tags to set default thumbnail 1 week ago
Christopher Usher b1f6d8d0de Fixed bug in authenticate_artist 2 weeks ago
Christopher Usher 1ba8597957 Made including a new image optional when updating a template 2 weeks ago
Christopher Usher 2c5e1058e8 Added missing authentication decorators on calls that modify the
templates database table
2 weeks ago
Christopher Usher 8020df8e60 Logging for thumbnail template management 3 weeks ago
Mike Lang d4d2bbcad4 thrimshim: Fix accepting thumbnail crop/location on submit 3 weeks ago
Mike Lang f4dc0ea7f1 thrimshim: Fix auth query again, hopefully properly this time 3 weeks ago
Mike Lang 696d1c8baa Fix typo in thrimshim auth SQL 3 weeks ago
Christopher Usher e0fc1eaf08 Refactor get_thumbnail and other fixes and improvments 3 weeks ago
Christopher Usher 70c8afe779 Switching thrimshim over to store thumbnail templates in the database 3 weeks ago
Christopher Usher 33f75efc19 Updatea thrimshim authentication to use roles table 3 weeks ago
ZeldaZach c378a1e4ab Add Audit Logging for several endpoints
- Use transactions for DB commits to avoid audit-less logs

Endpoints Supported:
- Manual Link
- Reset Row
- Update Row
2 months ago
Mike Lang 23ad78d592 Record in database when end time is "--"
We need this so that reverse sync reproduces these values correctly.

To handle this in the database, we have a composite type (dashed: boolean, value: timestamp).
Value is always valid and is equivalent to the old timestamp column,
but must be equal to start_time if dashed is true.

The only place we directly reference this column outside sheetsync is thrimshim, where we
always consider the value only.
3 months ago
Mike Lang 12150a4005 thrimshim: Return list of available transitions
For use in thrimbletrimmer for a drop-down.
3 months ago
Mike Lang a3aaa37bb0 Restructure playlists table
to have all tags, not just playlists.
Notably, this means playlist ids may be null.
3 months ago
Mike Lang 8dd6741ee9 schema: Playlist tags may be null, indicating "match nothing"
This is intended for partially-entered playlist data where tags have not been specified yet.
3 months ago
Mike Lang be111ccb2a Change database primary key from UUID to TEXT
We still store uuids, but in text form.
This allows us to store non-UUID ids for systems that have other ids.
3 months ago
Mike Lang 3606fadaa8 Pin gevent version to work around build issues
Seeing the following error on latest versions of gevent:

 Traceback (most recent call last):
   File "/usr/lib/python3.9/runpy.py", line 197, in _run_module_as_main
     return _run_code(code, main_globals, None,
   File "/usr/lib/python3.9/runpy.py", line 87, in _run_code
     exec(code, run_globals)
   File "/usr/lib/python3.9/site-packages/zulip_bots/schedulebot.py", line 2, in <module>
     import gevent.monkey
   File "/usr/lib/python3.9/site-packages/gevent/__init__.py", line 72, in <module>
     from gevent._hub_local import get_hub
   File "/usr/lib/python3.9/site-packages/gevent/_hub_local.py", line 150, in <module>
     import_c_accel(globals(), 'gevent.__hub_local')
   File "/usr/lib/python3.9/site-packages/gevent/_util.py", line 148, in import_c_accel
     mod = importlib.import_module(cname)
   File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
     return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named 'gevent._gevent_c_hub_local'
1 year ago
Mike Lang 86951d5916 thrimshim: Fix bug in extrapolate code when time is a float 1 year ago
Mike Lang 7a27938244 thrimshim: Expose time of day and 24h time 1 year ago
Mike Lang 78c053000e Upgrade pip in order to make wheels work 1 year ago
Mike Lang c436871ba6 Fix bug in thrimshim for archive cuts 1 year ago
Mike Lang 3b1af8c5e5 thrimshim: Expose clock readings 1 year ago
Mike Lang 720f1c4b41 thrimshim: Sanity checks for odometer 1 year ago
Mike Lang 06cd5d48ee More extrapolation fixes 1 year ago
Mike Lang 834dc55b41 thrimshim: Fix typo in odo extrapolation 1 year ago
Mike Lang f0e2aae519 thrimshim: Fix odometer endpoint 1 year ago
Mike Lang a83dbb58f7 thrimshim: Fix default templates 1 year ago
Mike Lang 734a7371f3 Add bus_data DB table and have thrimshim able to query it for latest odo reading 1 year ago
Mike Lang 8d984110ab thrimshim: Special handling for "archive" events 1 year ago
Mike Lang cbf3269001 thrimshim: Update default template names 1 year ago
Mike Lang 044dfb8084 Pin argh to avoid stupid breaking changes 1 year ago
Mike Lang 30d5ccc483 Fix all old references to github.com/ekimekim/wubloader 1 year ago
Mike Lang c36976dcca Default to rdp template for RDP thumbnails 2 years ago
ElementalAlchemist fafa2fa125 Add missing bracket to playlist strip regex 2 years ago
Mike Lang 6a713993af fix another typo 2 years ago
Mike Lang 8fbbb3f0a1 thrimshim: Fix description footer formatting
Needs newlines before playlists section, and a line between playlist section and footer.
2 years ago
Mike Lang 85c4319f2a thrimshim: Fix bug where description playlists header was added with zero playlists
We only want it to show at all if there's at least one playlist.
2 years ago
Mike Lang a298087137 playlists in description: round trip the description correctly 2 years ago
Mike Lang 17c5e86250 wip: 2 years ago
Mike Lang 34a33fdeb6 partially implement playlist links in video descriptions
We make them conceptually "part of the footer" so they're updated only when the video
is otherwise updated (which would generally mean MODIFIED).
2 years ago