Bladeren bron

tweak gifski

Mike L 3 jaren geleden
bovenliggende
commit
5bd9836601
4 gewijzigde bestanden met toevoegingen van 10 en 10 verwijderingen
  1. 3 3
      dist/gifski.js
  2. 2 2
      dist/wiki.js
  3. 3 3
      src/gifski.ts
  4. 2 2
      src/wiki.ts

+ 3 - 3
dist/gifski.js

@@ -13,7 +13,7 @@ const child_process_1 = require("child_process");
 const fs_1 = require("fs");
 const loggers_1 = require("./loggers");
 const logger = (0, loggers_1.getLogger)('gifski');
-const sizeLimit = 10 * Math.pow(2, 20);
+const sizeLimit = +Infinity;
 const roundToEven = (n) => Math.ceil(n / 2) * 2;
 exports.default = (inputFilePath, targetWidth) => __awaiter(void 0, void 0, void 0, function* () {
     const outputFilePath = inputFilePath.replace(/(?:\.[^.]*)?$/, '.gif');
@@ -23,10 +23,10 @@ exports.default = (inputFilePath, targetWidth) => __awaiter(void 0, void 0, void
             '-o',
             outputFilePath,
             '--fps',
-            '12.5',
+            '14.985',
             '--quiet',
             '--quality',
-            '90',
+            '96',
         ];
         if (typeof (targetWidth) === 'number') {
             args.push('--width', roundToEven(targetWidth).toString());

+ 2 - 2
dist/wiki.js

@@ -70,7 +70,7 @@ class default_1 {
                     const mediaFileName = `${filename}.${ext}`;
                     (0, fs_1.writeFileSync)(mediaFileName, Buffer.from(data));
                     return (ext === 'mp4' ?
-                        (0, gifski_1.default)(mediaFileName) :
+                        (0, gifski_1.default)(mediaFileName, 640) :
                         Promise.resolve(mediaFileName));
                 }
                 logger.warn('unable to find MIME type of fetched media, failing this fetch');
@@ -94,7 +94,7 @@ class default_1 {
                     const mediaPromise = this.fetchMedia(url)
                         .then(mediaFileName => {
                         const filename = `${fileNamePrefix}${indexOffset + index + 1}.${mediaFileName.split('.')[1]}`;
-                        logger.info(`uploading ${url} as ${filename}...`);
+                        logger.info(`uploading ${mediaFileName} as ${filename}...`);
                         return this.bot.simpleUpload({
                             file: mediaFileName,
                             filename,

+ 3 - 3
src/gifski.ts

@@ -5,7 +5,7 @@ import { getLogger } from './loggers';
 
 const logger = getLogger('gifski');
 
-const sizeLimit = 10 * 2 ** 20;
+const sizeLimit = +Infinity;
 const roundToEven = (n: number) => Math.ceil(n / 2) * 2;
 
 export default async (inputFilePath: string, targetWidth?: number) => {
@@ -16,10 +16,10 @@ export default async (inputFilePath: string, targetWidth?: number) => {
       '-o',
       outputFilePath,
       '--fps',
-      '12.5',
+      '14.985',
       '--quiet',
       '--quality',
-      '90',
+      '96',
     ];
     if (typeof(targetWidth) === 'number') {
       args.push('--width', roundToEven(targetWidth).toString());

+ 2 - 2
src/wiki.ts

@@ -83,7 +83,7 @@ export default class {
           const mediaFileName = `${filename}.${ext}`;
           writeFileSync(mediaFileName, Buffer.from(data));
           return (ext === 'mp4' ?
-            gifski(mediaFileName) :
+            gifski(mediaFileName, 640) :
             Promise.resolve(mediaFileName)
           );
         }
@@ -108,7 +108,7 @@ export default class {
           const mediaPromise = this.fetchMedia(url)
             .then(mediaFileName => {
               const filename = `${fileNamePrefix}${indexOffset + index + 1}.${mediaFileName.split('.')[1]}`;
-              logger.info(`uploading ${url} as ${filename}...`);
+              logger.info(`uploading ${mediaFileName} as ${filename}...`);
               return this.bot.simpleUpload({
                 file: mediaFileName,
                 filename,