|
|
xiota
|
|
I'd like to have it either, but apparently some redditors are... well, redditors.
|
|
2021-09-08 04:56:53
|
|
|
2021-09-08 04:58:01
|
The DL link on that page had expired, but I updated it for you. It's over a week old, but I haven't made any changes since the bug fixes.
|
|
2021-09-08 04:59:28
|
https://www.reddit.com/r/GIMP/comments/pcq7g9/jpeg_xl_gimp_plugin_for_windows/
|
|
|
|
veluca
|
2021-09-08 05:05:00
|
if some kind soul feels like figuring out how to get it to build in the CI, it would be great ๐
|
|
|
|
xiota
|
2021-09-08 05:05:16
|
how do you do that?
|
|
|
|
veluca
|
2021-09-08 05:05:32
|
you modify `.github/workflows/release.yaml`
|
|
|
|
xiota
|
2021-09-08 05:05:54
|
I can update the debian folder so that it builds the gimp plugin.
|
|
|
|
veluca
|
2021-09-08 05:05:57
|
then push to your `main` branch, see if the pipeline works, get the artifact, and see if the plugin works
|
|
2021-09-08 05:06:01
|
I'm doing that
|
|
2021-09-08 05:06:04
|
๐
|
|
2021-09-08 05:06:11
|
windows, not so familiar with...
|
|
|
|
xiota
|
2021-09-08 05:06:21
|
ah, I basically already have it done since I've been building on launchpad.
|
|
2021-09-08 05:06:37
|
i just need to copy and paste. unless you're already almost done with it.
|
|
|
|
veluca
|
2021-09-08 05:06:54
|
we probably want a different package though (and I need to also do it for the gdk plugin anyway)
|
|
2021-09-08 05:07:18
|
but if you want to do `libjxl-plugin-gimp` and then I do the gdk plugin, great for me ๐
|
|
2021-09-08 05:07:37
|
(you have been promoted to "dev", btw)
|
|
|
|
xiota
|
2021-09-08 05:08:13
|
haha... do i get any perks?
|
|
|
|
veluca
|
2021-09-08 05:08:27
|
recognizable color? ๐
|
|
|
|
xiota
|
2021-09-08 05:09:08
|
i had been calling it gimp-jxl... is libjxl-plugin-gimp preferred?
|
|
|
|
veluca
|
2021-09-08 05:09:24
|
or libjxl-gimp
|
|
2021-09-08 05:09:42
|
I have seen that gdk plugins use XXX-gdk-pixbuf
|
|
|
|
xiota
|
2021-09-08 05:14:02
|
gimp plugins look like gimp-* ... would it be better to follow the existing naming patterns or to have them all grouped together as libjxl-*
|
|
|
|
veluca
|
2021-09-08 05:14:54
|
I *think* it depends on where the source comes from
|
|
2021-09-08 05:15:08
|
if the source is in the libjxl tree, libjxl-...
|
|
2021-09-08 05:15:16
|
otherwise, if they are in the gimp tree, gimp-...
|
|
2021-09-08 05:15:19
|
I dunno though
|
|
2021-09-08 05:15:36
|
but if all gimp plugins are gimp-..., then I guess we can do gimp-...
|
|
|
|
xiota
|
2021-09-08 05:15:53
|
it looked to me like the gimp plugins were from all over the place, but i haven't looked closely.
|
|
2021-09-08 05:16:31
|
i think libjxl-gimp-plugin would be fine if you're using libjxl-gdk-pixbuf
|
|
|
|
veluca
|
2021-09-08 05:16:45
|
or even just libjxl-gimp
|
|
2021-09-08 05:16:49
|
eh, that or gimp-jxl
|
|
2021-09-08 05:16:53
|
up to you really
|
|
|
|
xiota
|
2021-09-08 05:16:55
|
haha
|
|
2021-09-08 05:18:10
|
i'll go with libjxl-gimp for now. it'll get reviewed and changed if needed.
|
|
2021-09-08 05:19:07
|
do you know if the `debian/tmp/` part in the install files is needed for debian? on launchpad they're not.
|
|
|
|
veluca
|
2021-09-08 05:22:29
|
no clue...
|
|
2021-09-08 05:22:46
|
I assume both work
|
|
|
|
xiota
|
2021-09-08 05:32:32
|
just opened a PR... will see if it builds.
|
|
2021-09-08 05:33:57
|
So where to download the artifacts that have been mentioned?
|
|
|
|
veluca
|
2021-09-08 05:34:19
|
ah, you can't from just the PR
|
|
2021-09-08 05:34:46
|
you need to push to your own main branch
|
|
2021-09-08 05:35:00
|
then in the pipeline runs you get the artifacts
|
|
|
|
xiota
|
2021-09-08 05:36:16
|
I see... seems like a lot of resources being used up...
|
|
2021-09-08 05:36:34
|
if people are doing their own personal builds just to get binaries.
|
|
|
|
veluca
|
2021-09-08 05:39:36
|
nah, you don't need to do it that way as an user
|
|
2021-09-08 05:39:45
|
it's just to test that it works
|
|
2021-09-08 05:40:24
|
you can use this to get binaries locally: https://artifacts.lucaversari.it/
|
|
2021-09-08 05:40:39
|
i.e. `deb https://artifacts.lucaversari.it/libjxl/libjxl/latest/deb/bionic /`
|
|
|
|
xiota
|
2021-09-08 05:48:55
|
do you know what build scripts that site uses?
|
|
|
spider-mario
|
2021-09-08 05:49:37
|
I would hope so
|
|
2021-09-08 05:50:20
|
(lucaversari -> **luca** **ve**rsari -> ve luca -> veluca)
|
|
|
BlueSwordM
|
2021-09-08 05:50:41
|
Actually, I have a good idea for the Gimp Windows plugin.
|
|
|
|
veluca
|
2021-09-08 05:50:43
|
I just get the artefacts from github ๐
|
|
|
BlueSwordM
|
2021-09-08 05:50:53
|
How about we convince the Windows maintainers to include it by default?
|
|
|
|
xiota
|
2021-09-08 05:51:09
|
hahaha
|
|
2021-09-08 05:51:56
|
the similarity in the names would have occurred to me eventually, i think.
|
|
|
BlueSwordM
|
|
BlueSwordM
How about we convince the Windows maintainers to include it by default?
|
|
2021-09-08 05:52:26
|
Having cross platform support of JXL stuff in Gimp would easily make adoption better, especially since Action! supports capturing JXL screenshots, so there is an easy way to get JXL stuff already.
|
|
|
|
veluca
|
|
BlueSwordM
How about we convince the Windows maintainers to include it by default?
|
|
2021-09-08 05:53:42
|
you won't get complaints from me ๐ but I think building our own doesn't hurt either
|
|
|
|
xiota
|
2021-09-08 06:05:58
|
windows maintainers of gimp? the version of the plugin in the repository has some bugs that need to be fixed... I have a PR... but it needs to be updated for some recent API changes. (JxlSetBasicInfo)
|
|
|
|
veluca
|
2021-09-08 06:19:16
|
https://github.com/libjxl/libjxl/pull/558 <- debian packaging for the GDK plugin
|
|
|
|
xiota
|
2021-09-08 06:28:50
|
The google cla bot failed on my PR.
|
|
|
|
veluca
|
2021-09-08 06:34:26
|
yeah I see that
|
|
2021-09-08 06:34:33
|
try a `@googlebot I consent`
|
|
2021-09-08 06:34:53
|
(in a comment)
|
|
2021-09-08 06:34:59
|
I think it will poke it in the right way
|
|
|
|
xiota
|
2021-09-08 06:36:03
|
Thanks.
|
|
|
|
veluca
|
2021-09-08 06:36:28
|
can I submit?
|
|
|
|
xiota
|
2021-09-08 06:36:42
|
Do i need to alert deymo for review?
|
|
|
|
veluca
|
2021-09-08 06:36:49
|
nah, he reviewed already
|
|
2021-09-08 06:37:03
|
I can approve again if need be
|
|
|
|
xiota
|
2021-09-08 06:37:12
|
I made all the changes suggested and all the builds passed, so unless anything else is needed, mine is ready.
|
|
|
|
veluca
|
2021-09-08 06:37:27
|
good, submitted
|
|
|
|
xiota
|
2021-09-08 06:37:30
|
thanks
|
|
|
|
veluca
|
2021-09-08 06:37:43
|
I'll rebase mine now
|
|
2021-09-08 06:38:16
|
in 1~2h the artifacts thingy should be updated ๐
|
|
|
|
xiota
|
2021-09-08 06:38:35
|
the gimp plugin has some bugs though
|
|
|
|
veluca
|
2021-09-08 06:42:16
|
I guess those will be fixed once we have your CL in?
|
|
|
|
xiota
|
2021-09-08 06:46:25
|
This PR needs to be reviewed... https://github.com/libjxl/libjxl/pull/525
|
|
2021-09-08 06:49:16
|
what is CL?
|
|
2021-09-08 06:53:33
|
so every build test, all the packages are redownloaded? isn't that heavy load on the ubuntu/debian/etc servers?
|
|
|
BlueSwordM
|
|
xiota
windows maintainers of gimp? the version of the plugin in the repository has some bugs that need to be fixed... I have a PR... but it needs to be updated for some recent API changes. (JxlSetBasicInfo)
|
|
2021-09-08 06:54:14
|
Basically, make the JXL plugin for Gimp become an integral part of Gimp when it works well with all of the UI options.
|
|
|
|
veluca
|
2021-09-08 06:55:03
|
CL = PR
|
|
2021-09-08 06:55:22
|
pretty sure github has its own debian mirrors ๐
|
|
|
BlueSwordM
Basically, make the JXL plugin for Gimp become an integral part of Gimp when it works well with all of the UI options.
|
|
2021-09-08 06:56:00
|
I don't know if they are planning to accept new plugins, the GIMP 3 plugin interface is super different AFAIU
|
|
2021-09-08 06:57:53
|
(and I don't know when GIMP 3 will be out)
|
|
|
|
xiota
|
2021-09-08 06:58:11
|
I expect 2025 or later.
|
|
2021-09-08 06:59:57
|
2.8 was released 2012-05-03. 2.10 was released 2018-10-14. So 6.5 years between releases... Next up is, 2025-03-xx
|
|
|
|
veluca
|
2021-09-08 07:01:24
|
heh
|
|
2021-09-08 07:01:35
|
I can believe that
|
|
|
|
xiota
|
2021-09-08 07:02:18
|
2.10 is pretty good though. I'm pretty happy with it.
|
|
2021-09-08 07:02:49
|
Just not going to chase after 3.0.
|
|
|
|
veluca
|
|
xiota
This PR needs to be reviewed... https://github.com/libjxl/libjxl/pull/525
|
|
2021-09-08 08:31:35
|
I reviewed it
|
|
2021-09-08 08:41:55
|
looks like Ubuntu 18.04 does *not* have Gimp 2.10... is there a reason why we couldn't build for Gimp 2.8?
|
|
|
spider-mario
|
2021-09-08 08:50:26
|
iirc, color management was very different (lacking) in 2.8
|
|
|
|
veluca
|
2021-09-08 08:51:47
|
Gegl wasn't a thing either, apparently
|
|
2021-09-08 08:51:53
|
ugh
|
|
2021-09-08 08:52:01
|
definitely no gimp 2.8
|
|
|
spider-mario
|
2021-09-08 08:52:07
|
https://ninedegreesbelow.com/photography/users-guide-to-high-bit-depth-gimp.html
|
|
2021-09-08 08:52:28
|
> GIMP 2.9/2.10 is a radical departure from GIMP 2.8, offering:
>
> - New precision options for high bit depth, radiometrically correct editing.
> - New color management options.
> - New and corrected color space decompositions, blend modes, and conversions to black and white.
> - Floating point precision with unclamped editing.
|
|
|
|
veluca
|
2021-09-08 09:03:39
|
ok, no gimp plugin for 18.04 ๐
|
|
|
|
xiota
|
2021-09-08 09:30:33
|
I've had problems building highway for 18.04 on launchpad. Haven't looked at it carefully to figure out what the problem is.
|
|
2021-09-08 09:31:12
|
Here's the build log: https://launchpadlibrarian.net/556401401/buildlog_ubuntu-bionic-amd64.highway_0.14.2-0xiota1_BUILDING.txt.gz
|
|
|
|
veluca
|
2021-09-08 09:32:36
|
lovely
|
|
2021-09-08 09:32:50
|
there's no "good" gtest package in 18.04
|
|
2021-09-08 09:32:57
|
our CI installs it manually I believe
|
|
2021-09-08 09:33:21
|
https://github.com/libjxl/libjxl/blob/main/.github/workflows/release.yaml#L179
|
|
2021-09-08 09:33:49
|
or something like that...
|
|
|
|
xiota
|
2021-09-08 09:35:09
|
That makes sense.
|
|
|
Petr
|
|
xiota
https://www.reddit.com/r/GIMP/comments/pcq7g9/jpeg_xl_gimp_plugin_for_windows/
|
|
2021-09-09 07:12:50
|
I've just tried it with GIMP 2.10.24 and it works pretty well. ๐ Thanks for all your Effort/Speed. ๐ I also commented on https://github.com/libjxl/libjxl/pull/525
|
|
2021-09-09 07:31:24
|
If a user moves the Distance slider to 0, would it make sense to check the "Lossless Mode" check box automatically (and vice versa)?
|
|
|
BlueSwordM
|
2021-09-09 02:17:12
|
Anyway, I have to say the new JXL plugin for Gimp is very good ๐
|
|
2021-09-09 02:17:16
|
I really like it.
|
|
|
|
xiota
|
2021-09-09 04:37:27
|
<@!321486891079696385> Are you using the current repository version or the Windows binaries I built?
|
|
|
BlueSwordM
|
|
xiota
<@!321486891079696385> Are you using the current repository version or the Windows binaries I built?
|
|
2021-09-09 04:37:51
|
The current repository version, as I mainly work on Linux and build from Got.
|
|
|
|
xiota
|
2021-09-09 04:38:59
|
If you want, you can try out an update: https://github.com/libjxl/libjxl/pull/525
|
|
|
Petr
If a user moves the Distance slider to 0, would it make sense to check the "Lossless Mode" check box automatically (and vice versa)?
|
|
2021-09-09 04:54:59
|
I changed it so that the lossless checkbox no longer disables the other sliders. Instead, when the slider is moved away from zero, the lossless box is unchecked. It's still possible to use distance 0 with lossless unchecked. What happens in that case is XYB is used, and file size is cut in half compared with using lossless (without XYB). (For one of my test images, 94MB vs 42MB.)
|
|
|
Petr
|
|
xiota
I changed it so that the lossless checkbox no longer disables the other sliders. Instead, when the slider is moved away from zero, the lossless box is unchecked. It's still possible to use distance 0 with lossless unchecked. What happens in that case is XYB is used, and file size is cut in half compared with using lossless (without XYB). (For one of my test images, 94MB vs 42MB.)
|
|
2021-09-10 05:09:18
|
Good. And will it be understandable for the users this way?
|
|
|
|
xiota
|
2021-09-10 06:04:49
|
No idea. Need users to use it to see how it goes.
|
|
2021-09-10 07:54:37
|
Built on MSYS2 with clang.
|
|