Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Love this little app. #37

Open
alecazam opened this issue May 6, 2023 · 2 comments
Open

Love this little app. #37

alecazam opened this issue May 6, 2023 · 2 comments

Comments

@alecazam
Copy link

alecazam commented May 6, 2023

I have an open-source encoder/decoder that I develop called kram and a viewer called kramv. But I haven't yet ported the kramv to Windows. I was trying out some of my files in PicoPixel, and thought I'd share a couple of issues and suggestions. You got ktx/dds support and BC7. So many of the other images viewers failed to handle all of these, so thanks for releasing this little viewer.

  1. Would be nice to be able to Open and drop folder(s) into the Files view. I do a recursive descent of sub-folders depending on modifier keys held or not. It does look like I can select and drop multiple files from a folder.
  2. The BC4/BC5_Snorm rendering is wrong. BC4/5_Unorm is correct. I use snorm for normals, and also reconstruct depth and display that.
  3. I map red/green/blue/alpha to 1/2/3/4 for intuition. PicoPixel uses 2/3/4/5, with 1 for restoring all channels.
  4. Pinch-zoom with a trackpad is reversed from what I'd expect. Spreading fingers out should zoom in.
  5. The mousewheel zoom is reversed too for my liking. Could see prefs for each of these.
  6. Don't change scale when hiding showing the file list. But initially scale to the window when opening an image if pref set.
  7. Need to be able to advance through 2d-array faces with same logic as cube/3d textures.
  8. Need astc support from ktx files.
  9. Need etc2 support, but this is disappearing. EAC_R/RG11 are useful though.
  10. Need ktx2 support
  11. Had a BC4 image, but the grayscale single channel preference didn't work and still displayed it as red. This is important to show these as gray.
@jaytaoko
Copy link
Contributor

jaytaoko commented May 7, 2023

Hello and thank you for your feedback!
To answer your points:

  1. Can do
  2. I will look into BC4/BC5_snorm
  3. Yes, there is some logic in red being assign to 1 and so on... The question is what key to restore all.
  4. I will add an option in the setting so users can invert the behaviour
  5. Same as 4
  6. Will look into it.
  7. Can do
  8. I will look into it.
  9. I will look into it.
  10. I will look into it.
  11. Probably a bug I need to fix.

Best regards.

@alecazam
Copy link
Author

alecazam commented May 13, 2023

Also when I open the same file from Explorer, the app keeps adding it to the file list as a new file. I was expecting the old one to refresh in response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants