Code Modernization

[jens-ox]

I initially only wanted to set up proper linting, but wasn't able to find a decent middle-ground without doing a full refactor.

The following things have been done:

• all files migrated to ES2015 classes
• code moved to ES Modules
• externally available file decoders used instead of inlining them in lib
• proper linting setup
• linting setup added to CI

I know this is more or less an un-reviewable PR - if it's preferred, I can also release this fork as a separate package.

Here is the latest run of the CI workflow.

[hanayik]

I personally don’t think it should be a separate package. The NiiVue team have tried our best to contribute back to the projects we depend on when we can, rather than releasing modified packages under a new scope. I think we can get this stuff merged in, but it would be good to get some feedback from @rii-mango about any unintended consequences in downstream Daikon software. From: Jens Ochsenmeier ***@***.***> Date: Wednesday, 22 November 2023 at 11:14 To: rii-mango/Daikon ***@***.***> Cc: Subscribed ***@***.***> Subject: [rii-mango/Daikon] wip (PR #57) I initially only wanted to set up proper linting, but wasn't able to find a decent middle-ground without doing a full refactor. The following things have been done: * all files migrated to ES2015 classes * code moved to ES Modules * externally available file decoders used instead of inlining them in lib * proper linting setup * linting setup added to CI I know this is more or less an un-reviewable PR - if it's preferred, I can also release this fork as a separate package.

…

________________________________ You can view, comment on, or merge this pull request online at: #57 Commit Summary * 0488901<0488901> wip * 3c4fae8<3c4fae8> fix up pipeline * ed20cf4<ed20cf4> cleanup * dd653f8<dd653f8> baseline to class * 0b2a99f<0b2a99f> simplify jpeg-ls File Changes (35 files<https://github.com/rii-mango/Daikon/pull/57/files>) * A .eslintignore<https://github.com/rii-mango/Daikon/pull/57/files#diff-a0fdacd2ade87c5238dde44378f574f7e123e669acdf8b740878b14b33b20275> (3) * A .eslintrc.cjs<https://github.com/rii-mango/Daikon/pull/57/files#diff-e1ce717e1179a0db14e90ec5374768a206651ca807db58352991eb7895ed7c9c> (25) * M .github/workflows/test.yml<https://github.com/rii-mango/Daikon/pull/57/files#diff-faff1af3d8ff408964a57b2e475f69a6b7c7b71c9978cccc8f471798caac2c88> (7) * M README.md<https://github.com/rii-mango/Daikon/pull/57/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5> (2) * D lib/charLS-DynamicMemory-browser.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-68bcaaa4ba46bf1ea61eb61caa774192ad7e02d95cc2598c43a29810162c3b2f> (64) * M lib/jpeg-baseline.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-e73956bdbc9d2aff0bc802199111e96446c07ad95db93b2a5fa5d11caa71cb3e> (1886) * M lib/jpeg-ls.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-2ea5197cc5b41cc2df89922740e8de2d68753237f4a08340c2730b52f502f884> (224) * D lib/jpx.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-7bc5f928039d4d96387ad46aa2d93aeda8ede9390e7af173e65eb9af0c4a9a9b> (4051) * M package-lock.json<https://github.com/rii-mango/Daikon/pull/57/files#diff-053150b640a7ce75eff69d1a22cae7f0f94ad64ce9a855db544dda0929316519> (4299) * M package.json<https://github.com/rii-mango/Daikon/pull/57/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519> (24) * M src/compression-utils.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-0abf30266f59b0761efacc8bbefca765f7d55888ca98b9614cc347a0dfa5bb46> (76) * M src/dictionary.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-519b5fdb4ac8376a93ff3678cf2b2c75c403a09ae87cbc2e12a44408800286c8> (7334) * M src/image.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-ad20a1a6b0c6cdb94f5aa0f680475c5d5f3f1b163b19be745fe52b6db38ad056> (2628) * M src/iterator.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-f9a8bbc278e59dcf85d06bc12d13b86f41f9f8e5462efa01fb4d2e6e99607bd4> (54) * M src/license.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-7e3c89f81ce1a426a2a29f6a1fff2e965dda035f709cf940cc3b102c440c83f0> (18) * M src/main.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-27653c212e1cfe533e4eb2f7d0d3f89604c9de48a09583b4cbbbcbd08a07da79> (59) * M src/orderedmap.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-6801cc0013ecd8d18ca326b9937b36d04bffce3a919046bd40f3203f7d0ce2af> (98) * M src/parser.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-54c345dc104dc19440f9c2482b7883df820e8b9b699fdd8fa07e2773e7197a29> (677) * M src/rle.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-11b61962e8dab690c5fbfc77249460245a13aefb17bf9f4cbbef56f070d1d9fd> (305) * M src/series.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-3f4f85c247c4180d0e2a3910a84a8f285bd708fa3f1d223d3d6c5353f84f80d0> (891) * M src/siemens.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-d729e5df3c9931c9d44f3ec2a6d154768842c330670311e5e0700192361d9d0a> (237) * M src/tag.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-17854fa3e8ac03fc7cea7badea45774da7cfa2fc63197b6cf0ff37ae9372e821> (1194) * M src/utilities.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-b95e1f8196cfb4870e186925275bfe6593587570eba52b841fcd1441b6900414> (442) * M tests/browser.html<https://github.com/rii-mango/Daikon/pull/57/files#diff-c885513113201bce241e7514580bbff118e26b6e94f119ac4d4165058768bf56> (2) * M tests/driver-deflated.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-084f2e4ac65b6d17ea86a3c221b1898dbc5b2b257b794a6c0e82805ea9f5b0ca> (58) * M tests/driver-explicit-big.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-abaa9d9e7ab90c413475e6c878e243f81982102cedf050f5c3a9424235fec968> (61) * M tests/driver-explicit-little.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-2d22dfd1d347f77c5c74e23b7e93c97721cba1f4dfe6ad4b560f4535f1a8fd1e> (61) * M tests/driver-implicit.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-8af7eda14ae1ce0c7b5145b29a104bdd1546e3265c8738d81ac82d3095ec421a> (61) * M tests/driver-jpeg-2000.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-61f9cacebfc8b8678dcab0f5c95e2fef6f9bff05340263f999059a71e445ede6> (68) * M tests/driver-jpeg-baseline-8bit.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-9ff519a4ac0cad704bc4f82751eb49cca44e958daff0095d73a032952bf0bf00> (69) * M tests/driver-jpeg-lossless-sel1.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-0b9f8e093c7d3789427cbefa16c3c31bffd0b63247559b6b0ecf6f443fc48cc5> (68) * M tests/driver-jpeg-ls.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-6c4f83b2dcc282d4f440e70fbe1ea6108f8531076a1144a999d6d12b802b38c1> (69) * M tests/driver-rle.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-bc3c84ea8e31f5dddf5f9bf590974be9806ef4186df070cf67f5da19c0d17c78> (68) * M tests/driver-series.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-1dcf142de88aba6c7d073daeca28345e0b46d4ee9ebed1e8382fa38593ed8b8b> (112) * M tests/driver-tags.js<https://github.com/rii-mango/Daikon/pull/57/files#diff-b66a2cdc8844f92c0b6f92b67a0d32df88285c45c3a473276ab04d4d135886b9> (78) Patch Links: * https://github.com/rii-mango/Daikon/pull/57.patch * https://github.com/rii-mango/Daikon/pull/57.diff — Reply to this email directly, view it on GitHub<#57>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABHZ3OBX47H55PKRGU2AQ2TYFXNBZAVCNFSM6AAAAAA7WBWFM6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGAYDMMJQHA3TGNA>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[rii-mango]

Hi @jens-ox - I inherited this account when the previous owner took a new job. I don't have much JS/ES experience and depend on contributions to keep things current. So I'll trust your and @hanayik opinions on this. Regarding downstream projects, that's Papaya which is using a fragile custom Papaya-Builder that needs to be replaced with something more modern. I can set Papaya to reference the previous version of Daikon if needed before a newer build system is set up.

[rii-mango]

Hi @jens-ox, thanks for the pull requests. I have accepted the one in the JPEGLosslessDecoderJS repository and updated the version to 2.1.0 in GitHub and npm. However, when I try to build using the PR for Daikon, I'm getting missing library errors for CharLS and jpeg2000. Do you know what's going wrong there?

[jens-ox]

Hi @rii-mango! How can I reproduce this? npm run build and npm run test both work fine for me.

[rii-mango]

@jens-ox I thought JPEGLosslessDecoderJS was working well, but apparently I didn't test it well enough. Some people have been chiming in on a couple of issues with it:
rii-mango/JPEGLosslessDecoderJS#18
#58

It looks like there is no build or release folder in the repository anymore. Is the intended change to use build/main.cjs as the release version of lossless.js? Or is it lossless.ts now?

[niks2060]

Approved after reviewing changes but need to resolved merge conflict and above comments.

[rii-mango]

@niks2060 - after @jens-ox's PR that updated JPEGLosslessDecoderJS to TypeScript, there is no longer a release folder and people are unable to figure out how to use it. Will this PR for Daikon be the same?

[jens-ox]

@rii-mango @niks2060 I wouldn't merge this and expect nothing to break - it's more or less a full refactor after all, and code produced by the TypeScript compiler won't be structurally identical to the code before. I'm sorry for not stating that in this PR (and the one in JPEGLosslessDecoderJS). In particular, I missed that users had the option to directly download the library from GitHub.

I'll open a PR for JPEGLosslessDecoderJS to make sure users can import it as expected.

Edit: Opened a PR for JPEGLosslessDecoderJS. I will update this PR tomorrow to ensure that the output files are where they're expected.

[niks2060]

@niks2060 - after @jens-ox's PR that updated JPEGLosslessDecoderJS to TypeScript, there is no longer a release folder and people are unable to figure out how to use it. Will this PR for Daikon be the same?

I think we need to work on global access like webpackUniversalModuleDefinition and I will try to add same in JPEGLosslessDecoderJS and give it a try.

[jens-ox]

I think we need to work on global access like webpackUniversalModuleDefinition and I will try to add same in JPEGLosslessDecoderJS and give it a try.

@niks2060 not sure what you mean by that. I would recommend sticking to ES Modules and only compile to UMD if absolutely needed.

[niks2060]

@jens-ox I have worked on cornerstone and they have implemented same way using UMD.

[jens-ox]

@niks2060 if that aligns with downstream usage of Daikon, then that might be a good choice. Still, aside from ES modules being the official module standard, using ES modules instead of UMD has many advantages. I can recommend Mozilla's deep dive into ES modules in case you're unfamiliar with them.