profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/catdad/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Kiril Vatev catdad @Accusoft New York, NY https://www.kirilv.com I make things

catdad/canvas-confetti 2076

🎉 on-demand confetti gun

catdad/electronmon 35

🖥 run, watch, and restart electron apps using magic

catdad/css-raw-loader 13

:foggy: CSS Raw loader module for Webpack

catdad/ColorJS 7

:pencil2: convert and calculate colors in JavaScript

catdad/chord-explorer 3

🎸 look up guitar and ukulele chords

catdad/electron-template 2

⚛ opinionated electron application template

catdad/awesome-remote-job 1

A curated list of awesome remote jobs and resources. Inspired by https://github.com/vinta/awesome-python

catdad/brackets-shebang-highligher 1

💢 i don't want to click buttons, you figure it out

catdad/brackets-theme-one-gray 1

:tropical_drink: the dark Brackets theme

catdad/awesome-electron 0

Useful resources for creating apps with Electron

issue commentcatdad-experiments/libheif-js

Support for darwin-arm64 architecture

I am not sure what you are asking for. This is a javascript module. There is no native component. It already works anywhere where a full javascript runtime exists.

noeRls

comment created time in 4 days

push eventcatdad-experiments/pantone-table

Kiril Vatev

commit sha 9bc0dc6209c68ecce274e94a307bf8395ea70802

updating something to do with being a module

view details

Kiril Vatev

commit sha 2ddf83dd50c419934da2a7dd733c3ac4b5e579e1

identifying that the module has no side-effects

view details

Kiril Vatev

commit sha 3406b9a567c05c1e82c66d13eb37679936da107f

adding readme content

view details

Kiril Vatev

commit sha d25f89502118496e7ffbc3614cdb4c62f65d7c21

adding files to publish and keywords

view details

push time in 5 days

push eventcatdad-experiments/pantone-table

Kiril Vatev

commit sha 2a9e902fa25b39497984ce024f7e0246d48b57c2

fixing copyright name

view details

Kiril Vatev

commit sha 9915a7c1b136ea03e00013e8845ed1a3835f60ea

adding editorconfig

view details

Kiril Vatev

commit sha dd6e4639b0cf2b7c8376eefb56e67778358e0a5c

adding build for package from various pantone sources

view details

push time in 6 days

create barnchcatdad-experiments/pantone-table

branch : main

created branch time in 7 days

created repositorycatdad-experiments/pantone-table

created time in 7 days

create barnchcatdad/electron-template

branch : #40-fix-config-race-condition

created branch time in 10 days

create barnchcatdad/eol-fix-stream

branch : temp

created branch time in 19 days

create barnchcatdad/eol-fix-stream

branch : custom-eol

created branch time in 19 days

pull request commentcatdad/eol-fix-stream

Add EOL option.

Unfortunately, I actually had to revert this. I found out after I merged it that it does have a bug in it and does not allow for the same functionality as the previous implementation.

The bug is in this line:

var endCrlf = /\r|\n$/;

This regular expression checks for a \r anywhere in the string or a \n at the end of the string. This can help illustrate that:

image

The previous code included behavior when either is at the end. Looking at code coverage, you can see that the else in that check is not covered. When I change the expression to match both being at the end (either /(\r|\n)$/ or /\r$|\n$/), all the tests time out and the stream never completes.

I didn't have time to investigate what the issue is. If you want, feel free to recreate this PR and fix the issue.

I guess if nothing else, we both learned that less code can have more bugs 😉.

btimby

comment created time in 19 days

push eventcatdad/eol-fix-stream

Kiril Vatev

commit sha f87ec96ce10f02fef8251c91c890aa0bc93e4fc6

bumping version to 1.1.0

view details

Kiril Vatev

commit sha d20ae013841afc544a322c03071a510b1f1bb54b

Revert "Merge pull request #1 from btimby/eol-option" This reverts commit 1e15ff8e1dddce99107257ead9f50b1243eb0d08.

view details

Kiril Vatev

commit sha 1b82d3c42505b55bde2904fa9170ee744e57e56a

bumping version to 1.1.1

view details

Kiril Vatev

commit sha 876a64576e4288ed7490ff94cea0edce161e630a

Merge branch 'master' into use-regular-transform-stream

view details

push time in 19 days

created tagcatdad/eol-fix-stream

tag1.1.1

:beginner: end all the things with lf

created time in 19 days

push eventcatdad/eol-fix-stream

Kiril Vatev

commit sha d20ae013841afc544a322c03071a510b1f1bb54b

Revert "Merge pull request #1 from btimby/eol-option" This reverts commit 1e15ff8e1dddce99107257ead9f50b1243eb0d08.

view details

Kiril Vatev

commit sha 1b82d3c42505b55bde2904fa9170ee744e57e56a

bumping version to 1.1.1

view details

push time in 19 days

created tagcatdad/eol-fix-stream

tag1.1.0

:beginner: end all the things with lf

created time in 19 days

push eventcatdad/eol-fix-stream

Kiril Vatev

commit sha f87ec96ce10f02fef8251c91c890aa0bc93e4fc6

bumping version to 1.1.0

view details

push time in 19 days

create barnchcatdad/eol-fix-stream

branch : use-regular-transform-stream

created branch time in 19 days

push eventcatdad/eol-fix-stream

Ben Timby

commit sha cd57f7673bea27a93b0853addde6181fcbb18329

Add debug script.

view details

Ben Timby

commit sha 6d94236c00a9e7286d884966dae768cd7d7cc203

Add test for new requirement.

view details

Ben Timby

commit sha f148abcb501c8a92f483623a9f2213bcb7912ecc

Satisfy new requirement.

view details

Ben Timby

commit sha 095dcc298f6923019866a1d399170777e2c2eeec

Remove debugger line and obsolete assertion.

view details

Ben Timby

commit sha ad191a9b7abf6e8153ecfad3ab0e4f4897a9785b

Eliminate ES6 parameter default syntax, fix test parameter passing.

view details

Kiril Vatev

commit sha bbf9fc89ce1fe54729166de9697017d448e0e1e3

Merge branch 'master' into eol-option

view details

Ben Timby

commit sha e96e7aa085af4fea6c931862d78d6e727801bcec

Merge branch 'eol-option' of github.com:btimby/eol-fix-stream into eol-option

view details

Kiril Vatev

commit sha ae5827d2fb4909d22d6baaef9daabd6aaa36da7d

fixing syntax issue

view details

Kiril Vatev

commit sha d1921de93b9ad16713bad9f0758e692628c88b3a

adding a more complete set of tests

view details

Kiril Vatev

commit sha c68f9d61c044f9b804862af553439b9baf9e7d8e

fixing missing array method issue in older versions of node

view details

Kiril Vatev

commit sha 5d4b54b3add59b218b1df7a843f3baafb98f743c

using an options object rather than a single string option

view details

Kiril Vatev

commit sha 5e1c827647d0fa749f8d419a6035105667d4aeb4

adding information about the module options to the readme

view details

Kiril Vatev

commit sha 1e15ff8e1dddce99107257ead9f50b1243eb0d08

Merge pull request #1 from btimby/eol-option Add a custom `eol` option to allow outputting a different line ending

view details

push time in 19 days

PR merged catdad/eol-fix-stream

Add EOL option.
  • Caller can now specify desired EOL sequence: '\r', '\n' or '\r\n'.
  • Simplify implementation.
+83 -29

7 comments

4 changed files

btimby

pr closed time in 19 days

pull request commentcatdad/eol-fix-stream

Add EOL option.

Sorry for the delay, I got super distracted. I made some additional changes and fixed some additional issues, and this looks good now. Thanks for adding this feature!

btimby

comment created time in 19 days

PullRequestReviewEvent

push eventbtimby/eol-fix-stream

Kiril Vatev

commit sha 5d4b54b3add59b218b1df7a843f3baafb98f743c

using an options object rather than a single string option

view details

Kiril Vatev

commit sha 5e1c827647d0fa749f8d419a6035105667d4aeb4

adding information about the module options to the readme

view details

push time in 19 days

push eventbtimby/eol-fix-stream

Kiril Vatev

commit sha c68f9d61c044f9b804862af553439b9baf9e7d8e

fixing missing array method issue in older versions of node

view details

push time in 19 days

push eventbtimby/eol-fix-stream

Kiril Vatev

commit sha ae5827d2fb4909d22d6baaef9daabd6aaa36da7d

fixing syntax issue

view details

Kiril Vatev

commit sha d1921de93b9ad16713bad9f0758e692628c88b3a

adding a more complete set of tests

view details

push time in 19 days

issue commentcatdad-experiments/heic-convert

5MB image take time almost 20 sec in AWS Lambda function

So let's break this down a little bit and hopefully this info will help:

First, it seems you are talking about timing an operation where only part of it is affected by this library. You are not really telling me whether this library is the problem, whether some other part of your function is the problem, or whether something about Lamba itself is a problem. I will really only attempt to help with issues using this library, as this is not the right forum for general software advice. Can you please make sure you are timing only using this library and provide that information?

Second, the size of the image alone is not super helpful. The size of the HEIC file itself and the size of the compressed content that is stored inside the HEIC can vary significantly based on many variables. The data that you are decoding is what is important. Would you be able to provide an example file that you are debugging this problem with?

Finally, this library is very likely solving a different problem than what you are solving. There are many ways to convert a HEIC image to another format. This library tries to solve the problem by converting the image in JavaScript, for users who are limited to using JavaScript. It does not try to solve the problem of converting it as fast as possible for users who do not have technology limitations. Since you are running in Lambda, you can use pretty much any tool. It might be best to find the tool that tries to solve the problem the fastest rather than a tool that is most convenient due to solving the problem with only JavaScript.

bhandurejitu

comment created time in 19 days

issue closedcatdad-experiments/heic-convert

how can i use this code in php

is there any repo/composer for php?

closed time in 20 days

bhandurejitu

issue commentcatdad-experiments/heic-convert

how can i use this code in php

I don't know. Sorry, this is a NodeJS project and I have no experience using PHP.

bhandurejitu

comment created time in 20 days

starteddanvk/source-map-explorer

started time in 21 days

startedevmar/webtreemap

started time in 21 days

pull request commentcatdad/video-tools-app

Bump path-parse from 1.0.6 to 1.0.7

@dependabot recreate

dependabot[bot]

comment created time in 25 days

delete branch catdad/video-tools-app

delete branch : dependencies

delete time in 25 days

push eventcatdad/video-tools-app

Kiril Vatev

commit sha daac24871927baa9e73529b8ce04056907f92b17

updating various dev dependencies

view details

Kiril Vatev

commit sha 6f369ed3f6682e7039da610c8ec4a965f3723f62

updating various dependencies

view details

Kiril Vatev

commit sha 34cc0d2ba6446c2e97333b9d17ab23c2beaa7deb

getting rid of using remote, updating to electron 12

view details

Kiril Vatev

commit sha 46430a5e313a1bfa2b194a6aaa5b7ebef8d2f1cd

making sure that reads and writes from autosaving and writing config can't overlap and cause conflicts

view details

Kiril Vatev

commit sha 57f97155c369c2a776d41d97f230a38940a0fdf3

removing use of PQueue (in line with electron-template), making sure that config is persisted correctly

view details

Kiril Vatev

commit sha 72c8798512c3a0c2a7d1c4b2f26a2454ad482148

Merge pull request #20 from catdad/dependencies updating various dependencies, removing `remote` module use

view details

push time in 25 days