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

Potential inset orientation issue. #865

Open
bryanedds opened this issue Sep 6, 2024 · 4 comments
Open

Potential inset orientation issue. #865

bryanedds opened this issue Sep 6, 2024 · 4 comments
Labels
breaking changes APIs that are more likely to affect users discussion investigation a topic involving investigation

Comments

@bryanedds
Copy link
Owner

bryanedds commented Sep 6, 2024

Currently, it appears that 2D inset values reference texels with top-left as origin whereas it would make more sense in many cases (and be more consistent with other systems) with bottom-left as origin. Let's consider modifying the sprite rendering pipeline to change this.

@bryanedds bryanedds added discussion investigation a topic involving investigation breaking changes APIs that are more likely to affect users labels Sep 6, 2024
@bryanedds
Copy link
Owner Author

bryanedds commented Sep 6, 2024

The only nice thing I can say about how it currently works is that it matches how nearly all paint / image editing programs work, as well as tile editing programs such as Tiled. So there is still a sort of consistency here even thought it's overall inconsistent with how Nu does other spaces.

@bryanedds
Copy link
Owner Author

CC'ing mapeditor/tiled#249 (comment) since this is the issue that once again brought the subject to the forefront of my mind.

@bryanedds
Copy link
Owner Author

bryanedds commented Sep 6, 2024

Maybe we just accept that this will be a problem until the end of time? 2D authoring programs will always have their Y's inverted compared to engine representations. Included in that space are also tile editing programs like Tiled. This inconsistent frame representation is pretty much accepted as-is in the 3D authoring and engine space. However, it's a real problem because even to this day, there is a minor enhancement I'd like to add to Omni Blade's map generation code that I just can't wrap my head around due to this aspect causing a little too much cognitive load for me to carry comfortably - and thus remains unimplemented.

@bryanedds
Copy link
Owner Author

bryanedds commented Oct 17, 2024

The best way I can describe this phenomenon is that inset is currently in 'authoring space' or 'paint program space'. In terms of the engine itself, it seems weird and inconsistent. But in terms of using it in tandem with image authoring, it can be a lot more intuitive.

We still don't have enough information about which way to go to take action on this issue, so it remains unclosed.

@bryanedds bryanedds mentioned this issue Nov 14, 2024
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking changes APIs that are more likely to affect users discussion investigation a topic involving investigation
Projects
None yet
Development

No branches or pull requests

1 participant